Skip to content

[multistage] fix literal aggregate function not working issue#9331

Merged
walterddr merged 2 commits intoapache:masterfrom
walterddr:pr_fix_count
Sep 3, 2022
Merged

[multistage] fix literal aggregate function not working issue#9331
walterddr merged 2 commits intoapache:masterfrom
walterddr:pr_fix_count

Conversation

@walterddr
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 2, 2022

Codecov Report

Merging #9331 (8ee7571) into master (6047b06) will increase coverage by 40.20%.
The diff coverage is 100.00%.

❗ Current head 8ee7571 differs from pull request most recent head 804f328. Consider uploading reports for the commit 804f328 to get more accurate results

@@              Coverage Diff              @@
##             master    #9331       +/-   ##
=============================================
+ Coverage     28.37%   68.57%   +40.20%     
- Complexity       53     5046     +4993     
=============================================
  Files          1867     1879       +12     
  Lines         99608    99971      +363     
  Branches      15167    15208       +41     
=============================================
+ Hits          28260    68556    +40296     
+ Misses        68608    26505    -42103     
- Partials       2740     4910     +2170     
Flag Coverage Δ
integration1 26.14% <0.00%> (+0.02%) ⬆️
integration2 ?
unittests1 67.11% <100.00%> (?)
unittests2 15.35% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...inot/query/runtime/operator/AggregateOperator.java 84.37% <100.00%> (+84.37%) ⬆️
...t/core/plan/StreamingInstanceResponsePlanNode.java 0.00% <0.00%> (-100.00%) ⬇️
...ore/operator/streaming/StreamingResponseUtils.java 0.00% <0.00%> (-100.00%) ⬇️
...server/starter/helix/SegmentReloadStatusValue.java 0.00% <0.00%> (-100.00%) ⬇️
.../operator/blocks/results/MetadataResultsBlock.java 0.00% <0.00%> (-100.00%) ⬇️
...ager/realtime/PeerSchemeSplitSegmentCommitter.java 0.00% <0.00%> (-100.00%) ⬇️
...urces/ServerReloadControllerJobStatusResponse.java 0.00% <0.00%> (-100.00%) ⬇️
...ator/streaming/StreamingSelectionOnlyOperator.java 0.00% <0.00%> (-90.00%) ⬇️
...he/pinot/core/plan/StreamingSelectionPlanNode.java 0.00% <0.00%> (-88.89%) ⬇️
...che/pinot/common/auth/StaticTokenAuthProvider.java 0.00% <0.00%> (-87.50%) ⬇️
... and 1349 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@walterddr walterddr merged commit b4c8271 into apache:master Sep 3, 2022
@Jackie-Jiang Jackie-Jiang added the multi-stage Related to the multi-stage query engine label Sep 13, 2022
@walterddr walterddr deleted the pr_fix_count branch December 6, 2023 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

multi-stage Related to the multi-stage query engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants