Skip to content

Conversation

@xiangfu0
Copy link
Contributor

@xiangfu0 xiangfu0 commented Jul 29, 2023

Support multi-value aggregation functions:

  • countMV
  • sumMV
  • minMV
  • maxMV
  • avgMV
  • minMaxRangeMV
  • distinctCountMV
  • distinctCountBitmapMV
  • distinctCountHLLMV
  • distinctSumMV
  • distinctAvgMV
  • percentileMV
  • percentileEstMV
  • percentileTDigestMV
  • percentileKLLMV

@xiangfu0 xiangfu0 changed the title V2 mv aggregation [multistage] Support multi-value aggregation functions Jul 29, 2023
@xiangfu0 xiangfu0 requested a review from walterddr July 29, 2023 00:30
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2023

Codecov Report

Merging #11216 (44c8682) into master (0598955) will decrease coverage by 0.01%.
Report is 13 commits behind head on master.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #11216      +/-   ##
==========================================
- Coverage    0.11%    0.11%   -0.01%     
==========================================
  Files        2227     2229       +2     
  Lines      119594   119803     +209     
  Branches    18099    18126      +27     
==========================================
  Hits          137      137              
- Misses     119437   119646     +209     
  Partials       20       20              
Flag Coverage Δ
integration1temurin11 0.00% <0.00%> (ø)
integration1temurin17 0.00% <0.00%> (ø)
integration1temurin20 0.00% <0.00%> (ø)
integration2temurin11 0.00% <0.00%> (ø)
integration2temurin17 0.00% <0.00%> (?)
integration2temurin20 0.00% <0.00%> (ø)
unittests1temurin11 0.00% <0.00%> (ø)
unittests1temurin17 0.00% <0.00%> (ø)
unittests1temurin20 0.00% <0.00%> (ø)
unittests2temurin11 0.11% <0.00%> (-0.01%) ⬇️
unittests2temurin17 0.11% <0.00%> (-0.01%) ⬇️
unittests2temurin20 0.11% <0.00%> (-0.01%) ⬇️

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

Files Changed Coverage Δ
...che/pinot/segment/spi/AggregationFunctionType.java 0.00% <0.00%> (ø)

... and 23 files with indirect coverage changes

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

@xiangfu0 xiangfu0 force-pushed the v2-mv-aggregation branch 2 times, most recently from b3046a6 to 5ee6ad0 Compare July 29, 2023 05:14
@xiangfu0 xiangfu0 requested a review from Jackie-Jiang July 29, 2023 05:38
@xiangfu0 xiangfu0 force-pushed the v2-mv-aggregation branch from 5ee6ad0 to a40a886 Compare July 29, 2023 06:26
@Jackie-Jiang Jackie-Jiang added feature multi-stage Related to the multi-stage query engine labels Jul 31, 2023
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we add unit-tests in the json resources?

@xiangfu0 xiangfu0 force-pushed the v2-mv-aggregation branch from a40a886 to 44c8682 Compare August 1, 2023 06:48
@xiangfu0 xiangfu0 merged commit 9d7676e into apache:master Aug 1, 2023
@xiangfu0 xiangfu0 deleted the v2-mv-aggregation branch August 1, 2023 09:52
s0nskar pushed a commit to s0nskar/pinot that referenced this pull request Aug 10, 2023
* temp

* Support multi-value aggregation functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature multi-stage Related to the multi-stage query engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants