Skip to content

fix: missing table stats make performance awful? #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 13, 2025
Merged

fix: missing table stats make performance awful? #71

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 16 additions & 11 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,11 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [4.0.8] - Unreleased

### Fixed
- #70: Performance regression on newer IRIS versions when table stats are missing on a clean instance/run

## [4.0.7] - 2025-09-23

### Fixed
Expand All @@ -18,48 +23,48 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [4.0.5] - 2024-11-04

### Fixed
### Fixed
- #57: Improve SQL performance when mapping run coverage

## [4.0.4] - 2024-10-15

### Fixed
### Fixed
- #54: Defend against possible configuration-dependent SQL exceptions in mapping INT to MAC/CLS coverage

## [4.0.3] - 2024-08-19

### Fixed
- #52: Method mapping code now doesn't use AST's endline_no property to support older python versions
### Fixed
- #52: Method mapping code now doesn't use AST's endline_no property to support older python versions
- #53: Ignore traced commands from code without a class name

## [4.0.2] - 2024-08-16

### Fixed
- #51: Don't start (and stop) the ObjectScript and Python monitors if there are no ObjectScript/Python routines being tracked respectively, fixes error from trying to start/stop the %Monitor.System.LineByLine with no routines
- #51: Don't start (and stop) the ObjectScript and Python monitors if there are no ObjectScript/Python routines being tracked respectively, fixes error from trying to start/stop the %Monitor.System.LineByLine with no routines


## [4.0.1] - 2024-08-16

### Fixed
- #45: Fixed Python line 0 tracking for 2024.2
### Fixed
- #45: Fixed Python line 0 tracking for 2024.2
- #46: Fix for bug caused by UpdateComplexity calling GetCurrentByName unnecessarily and causing dependency issues
- #47: Fixed mapping issue caused by empty lines at top of Python method not showing up in compiled Python
- #48: When the Line-By-Line Monitor resumes after pausing, resume the Python tracer too
- #48: When the Line-By-Line Monitor resumes after pausing, resume the Python tracer too
- #49: Added user parameter for preloading python modules (fixes problem of pandas breaking sys.settrace on first import)

## [4.0.0] - 2024-08-01

### Changed
- #29: As a consequence of this change, the minimum supported platform version is 2022.1

### Added
### Added
- #29: Track code coverage for embedded python methods in .cls files
- #42: Added a listener interface and manager with an associated user parameter, allowing the user to broadcast output on test method/case/suite completion.
- #42: Added a listener interface and manager with an associated user parameter, allowing the user to broadcast output on test method/case/suite completion.

## [3.1.1] - 2024-07-31

### Fixed
- #39: Fixed bug where results viewer gave divide by zero error when there were 0 executed methods in the covered code
- #39: Fixed bug where results viewer gave divide by zero error when there were 0 executed methods in the covered code
- #41: Now the code strips leading and trailing whitespace from coverage.list, so "PackageName.PKG " will still be loaded properly

## [3.1.0] - 2024-07-05
Expand Down
6 changes: 6 additions & 0 deletions cls/TestCoverage/Data/Run.cls
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,10 @@ ClassMethod MapRunCoverage(pRunIndex As %Integer) As %Status
Try {
Set tRun = ##class(TestCoverage.Data.Run).%OpenId(pRunIndex,,.tSC)
$$$ThrowOnError(tSC)

do $SYSTEM.SQL.Stats.Table.GatherSchemaStats("TestCoverage_Data","TuneLog.txt")
do $SYSTEM.SQL.Stats.Table.GatherSchemaStats("TestCoverage_Data_Aggregate","TuneLog.txt")
do $SYSTEM.SQL.Stats.Table.GatherSchemaStats("TestCoverage_Data_CodeSubUnit","TuneLog.txt")

// It would be wonderful if there was support for something along the lines of (with a few made up non-functions):
/*
Expand Down Expand Up @@ -117,6 +121,8 @@ ClassMethod MapRunCoverage(pRunIndex As %Integer) As %Status
Set tCoverage.CoveredLines = $BitLogic(tCoverage.CoveredLines|hCoveredLines)
$$$ThrowOnError(tCoverage.%Save())
}

do $SYSTEM.SQL.Stats.Table.GatherTableStats("TestCoverage_Data.Coverage","TuneLog.txt")

// Copy any other metrics captured/requested as well.
For i=1:1:tRun.Metrics.Count() {
Expand Down
2 changes: 1 addition & 1 deletion module.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<Export generator="Cache" version="25">
<Document name="TestCoverage.ZPM"><Module>
<Name>TestCoverage</Name>
<Version>4.0.7</Version>
<Version>4.0.8</Version>
<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>
<Packaging>module</Packaging>
<Resource Name="TestCoverage.PKG" Directory="cls" />
Expand Down
Loading