Skip to content

Commit 93114d8

Browse files
committed
fix: missing table stats make performance awful?
1 parent d93cab3 commit 93114d8

File tree

3 files changed

+11
-1
lines changed

3 files changed

+11
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
55
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
77

8+
## [3.1.2] - Unreleased
9+
10+
### Fixed
11+
- Performance regression on newer IRIS versions when table stats are missing on a clean instance/run
812

913
## [3.1.1] - 2024-07-31
1014

cls/TestCoverage/Data/Run.cls

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@ ClassMethod MapRunCoverage(pRunIndex As %Integer) As %Status
3535
Try {
3636
Set tRun = ##class(TestCoverage.Data.Run).%OpenId(pRunIndex,,.tSC)
3737
$$$ThrowOnError(tSC)
38+
39+
do $SYSTEM.SQL.Stats.Table.GatherSchemaStats("TestCoverage_Data","TuneLog.txt")
40+
do $SYSTEM.SQL.Stats.Table.GatherSchemaStats("TestCoverage_Data_Aggregate","TuneLog.txt")
41+
do $SYSTEM.SQL.Stats.Table.GatherSchemaStats("TestCoverage_Data_CodeSubUnit","TuneLog.txt")
3842

3943
// It would be wonderful if there was support for something along the lines of (with a few made up non-functions):
4044
/*
@@ -106,6 +110,8 @@ ClassMethod MapRunCoverage(pRunIndex As %Integer) As %Status
106110
Set tCoverage.CoveredLines = $BitLogic(tCoverage.CoveredLines|hCoveredLines)
107111
$$$ThrowOnError(tCoverage.%Save())
108112
}
113+
114+
do $SYSTEM.SQL.Stats.Table.GatherTableStats("TestCoverage_Data.Coverage","TuneLog.txt")
109115

110116
// Copy any other metrics captured/requested as well.
111117
For i=1:1:tRun.Metrics.Count() {

module.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<Export generator="Cache" version="25">
33
<Document name="TestCoverage.ZPM"><Module>
44
<Name>TestCoverage</Name>
5-
<Version>3.1.1</Version>
5+
<Version>3.1.2</Version>
66
<Description>Run your typical ObjectScript %UnitTest tests and see which lines of your code are executed. Includes Cobertura-style reporting for use in continuous integration tools.</Description>
77
<Packaging>module</Packaging>
88
<Resource Name="TestCoverage.PKG" Directory="cls" />

0 commit comments

Comments
 (0)