Conversation
commit: |
size-limit report 📦
|
There was a problem hiding this comment.
Pull Request Overview
This PR reduces memory usage by replacing full sample arrays with sample counts in the Statistics interface, addressing issue #61 related to data retention.
Key Changes:
- Modified
Statisticsinterface to store onlysamplesCount: numberinstead ofsamples: number[] - Updated
getStatisticsSorted()to return sample count instead of the full array - Updated all tests and console table formatting to use
samplesCount
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/types.ts | Changed Statistics interface to replace samples array with samplesCount number |
| src/utils.ts | Updated getStatisticsSorted() to return samplesCount: samples.length and console table converter to use new property |
| src/task.ts | Added type annotation for throughputSamples and GC-related assignments (though the latter are not functionally necessary) |
| test/statistics.test.ts | Updated test assertions to check samplesCount type instead of array validation |
| test/concurrency-iteration-limit.test.ts | Updated test assertions to check samplesCount value instead of array length |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Uzlopak
changed the title
retainSamples option on bench and task level
Uzlopak
changed the title
retainSamples option on bench and task levelretainSamples option on bench and task level, default is no samples retention
|
I made the data retention now configurable Do you have arguments against no data retention by default? |
Uzlopak
changed the title
retainSamples option on bench and task level, default is no samples retentionretainSamples option on bench and task level, default is samples retention
Uzlopak
changed the title
retainSamples option on bench and task level, default is samples retentionretainSamples option on bench and task level, default is true
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uzlopak
force-pushed
the
reduce-data-retention
branch
from
7bbb272 to
ea031f3
Compare
Uzlopak
changed the title
retainSamples option on bench and task level, default is trueretainSamples option on bench and task level, default is true
fixes #61