Skip to content

Fix the repeat planning for multi-stage query#11215

Merged
Jackie-Jiang merged 1 commit intoapache:masterfrom
Jackie-Jiang:fix_repeat_planning
Jul 29, 2023
Merged

Fix the repeat planning for multi-stage query#11215
Jackie-Jiang merged 1 commit intoapache:masterfrom
Jackie-Jiang:fix_repeat_planning

Conversation

@Jackie-Jiang
Copy link
Contributor

Fix the bug where we plan the query twice

@Jackie-Jiang Jackie-Jiang requested a review from walterddr July 29, 2023 00:19
@Jackie-Jiang Jackie-Jiang added the multi-stage Related to the multi-stage query engine label Jul 29, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2023

Codecov Report

Merging #11215 (078df12) into master (0598955) will increase coverage by 0.00%.
Report is 1 commits behind head on master.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master   #11215   +/-   ##
=======================================
  Coverage    0.11%    0.11%           
=======================================
  Files        2227     2227           
  Lines      119594   119592    -2     
  Branches    18099    18099           
=======================================
  Hits          137      137           
+ Misses     119437   119435    -2     
  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 Δ
.../java/org/apache/pinot/query/QueryEnvironment.java 0.00% <0.00%> (ø)

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

@Jackie-Jiang Jackie-Jiang merged commit 44434cf into apache:master Jul 29, 2023
@Jackie-Jiang Jackie-Jiang deleted the fix_repeat_planning branch July 29, 2023 02:49
s0nskar pushed a commit to s0nskar/pinot that referenced this pull request Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants