Closed grecinto closed 8 months ago
Attention: 26 lines
in your changes are missing coverage. Please review.
Comparison is base (
827f325
) 71.58% compared to head (1982265
) 70.69%. Report is 4 commits behind head on master.:exclamation: Current head 1982265 differs from pull request most recent head 37ed54e. Consider uploading reports for the commit 37ed54e to get more accurate results
Files | Patch % | Lines |
---|---|---|
btree/btree.go | 43.90% | 18 Missing and 5 partials :warning: |
btree/store_info.go | 0.00% | 2 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The Code coverage report is misleading, as the existing Unit & Integration tests are exercising indirectly even the new code that codecov did not catch.
Both features mentioned above were seen operational in unit & integration tests. And a lot faster/optimal now.
This improvement reduced I/O by a lot by preventing unnecessary delete and re-insert.