codecov-commenter commented on PR #16878: URL: https://github.com/apache/pinot/pull/16878#issuecomment-3349470856
## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/16878?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report :x: Patch coverage is `72.97688%` with `187 lines` in your changes missing coverage. Please review. :white_check_mark: Project coverage is 56.40%. Comparing base ([`9dda121`](https://app.codecov.io/gh/apache/pinot/commit/9dda1211fa2422e8cc89cbc735db002a1f735140?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`4b81e65`](https://app.codecov.io/gh/apache/pinot/commit/4b81e656cc76a64128d7e6fc0f89ea1916ad1938?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)). :warning: Report is 37 commits behind head on master. | [Files with missing lines](https://app.codecov.io/gh/apache/pinot/pull/16878?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines | |---|---|---| | [...y/aggregation/groupby/LongGroupByResultHolder.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcore%2Fquery%2Faggregation%2Fgroupby%2FLongGroupByResultHolder.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9xdWVyeS9hZ2dyZWdhdGlvbi9ncm91cGJ5L0xvbmdHcm91cEJ5UmVzdWx0SG9sZGVyLmphdmE=) | 32.25% | [16 Missing and 5 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...processing/genericrow/AdaptiveSizeBasedWriter.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcore%2Fsegment%2Fprocessing%2Fgenericrow%2FAdaptiveSizeBasedWriter.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9zZWdtZW50L3Byb2Nlc3NpbmcvZ2VuZXJpY3Jvdy9BZGFwdGl2ZVNpemVCYXNlZFdyaXRlci5qYXZh) | 32.25% | [19 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...he/pinot/core/operator/BitmapDocIdSetOperator.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcore%2Foperator%2FBitmapDocIdSetOperator.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9CaXRtYXBEb2NJZFNldE9wZXJhdG9yLmphdmE=) | 40.74% | [14 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...e/pinot/core/operator/ReverseDocIdSetOperator.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcore%2Foperator%2FReverseDocIdSetOperator.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9vcGVyYXRvci9SZXZlcnNlRG9jSWRTZXRPcGVyYXRvci5qYXZh) | 67.44% | [10 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [.../common/utils/config/QueryWorkloadConfigUtils.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcommon%2Futils%2Fconfig%2FQueryWorkloadConfigUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9waW5vdC9jb21tb24vdXRpbHMvY29uZmlnL1F1ZXJ5V29ya2xvYWRDb25maWdVdGlscy5qYXZh) | 85.55% | [4 Missing and 9 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [.../apache/pinot/common/audit/AuditServiceBinder.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcommon%2Faudit%2FAuditServiceBinder.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9waW5vdC9jb21tb24vYXVkaXQvQXVkaXRTZXJ2aWNlQmluZGVyLmphdmE=) | 0.00% | [8 Missing :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...ery/planner/partitioning/HashFunctionSelector.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-query-planner%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fquery%2Fplanner%2Fpartitioning%2FHashFunctionSelector.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtcXVlcnktcGxhbm5lci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvcXVlcnkvcGxhbm5lci9wYXJ0aXRpb25pbmcvSGFzaEZ1bmN0aW9uU2VsZWN0b3IuamF2YQ==) | 66.66% | [6 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...ggregation/function/SumIntAggregationFunction.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcore%2Fquery%2Faggregation%2Ffunction%2FSumIntAggregationFunction.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9xdWVyeS9hZ2dyZWdhdGlvbi9mdW5jdGlvbi9TdW1JbnRBZ2dyZWdhdGlvbkZ1bmN0aW9uLmphdmE=) | 90.78% | [2 Missing and 5 partials :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...he/pinot/spi/accounting/WorkloadBudgetManager.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-spi%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fspi%2Faccounting%2FWorkloadBudgetManager.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3Qtc3BpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9waW5vdC9zcGkvYWNjb3VudGluZy9Xb3JrbG9hZEJ1ZGdldE1hbmFnZXIuamF2YQ==) | 0.00% | [6 Missing and 1 partial :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...e/query/aggregation/LongAggregateResultHolder.java](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&filepath=pinot-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fcore%2Fquery%2Faggregation%2FLongAggregateResultHolder.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3QtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3QvY29yZS9xdWVyeS9hZ2dyZWdhdGlvbi9Mb25nQWdncmVnYXRlUmVzdWx0SG9sZGVyLmphdmE=) | 50.00% | [6 Missing :warning: ](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | ... and [29 more](https://app.codecov.io/gh/apache/pinot/pull/16878?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | > :exclamation: There is a different number of reports uploaded between BASE (9dda121) and HEAD (4b81e65). Click for more details. > > <details><summary>HEAD has 36 uploads less than BASE</summary> > >| Flag | BASE (9dda121) | HEAD (4b81e65) | >|------|------|------| >|java-21|5|1| >|unittests1|2|1| >|unittests|4|1| >|temurin|10|1| >|java-11|5|0| >|unittests2|2|0| >|integration|6|0| >|integration1|2|0| >|custom-integration1|2|0| >|integration2|2|0| ></details> <details><summary>Additional details and impacted files</summary> ```diff @@ Coverage Diff @@ ## master #16878 +/- ## ============================================ - Coverage 63.38% 56.40% -6.99% + Complexity 1409 701 -708 ============================================ Files 3063 2407 -656 Lines 179781 133871 -45910 Branches 27521 21270 -6251 ============================================ - Hits 113960 75509 -38451 + Misses 57029 52069 -4960 + Partials 8792 6293 -2499 ``` | [Flag](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [custom-integration1](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [integration](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [integration1](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [integration2](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [java-11](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [java-21](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `56.40% <72.97%> (-6.96%)` | :arrow_down: | | [temurin](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `56.40% <72.97%> (-6.99%)` | :arrow_down: | | [unittests](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `56.40% <72.97%> (-6.99%)` | :arrow_down: | | [unittests1](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `56.40% <72.97%> (+0.02%)` | :arrow_up: | | [unittests2](https://app.codecov.io/gh/apache/pinot/pull/16878/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more. </details> [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/pinot/pull/16878?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). <details><summary> :rocket: New features to boost your workflow: </summary> - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges. </details> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
