You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "xiangfu0 (via GitHub)" <gi...@apache.org> on 2026/04/15 17:06:52 UTC

[I] Re-enable QueryCompilationTest.testAggregateJoinRemove [pinot]

xiangfu0 opened a new issue, #18224:
URL: https://github.com/apache/pinot/issues/18224

   ### Motivation
   `pinot-query-planner/src/test/java/org/apache/pinot/query/QueryCompilationTest.java` still has `@Ignore` on `testAggregateJoinRemove()`. The ignore message says the test only passes when `PRUNE_RULES` run before `BASIC_RULES`, but enabling that changes roughly 50 hard-coded plans and may introduce redundant `Project` and `Exchange` nodes.
   
   ### Scope
   - Reproduce the failing planner state.
   - Decide whether the rule ordering, the expected plan, or the optimization itself is wrong.
   - Re-enable the test once the intended behavior is explicit.
   
   ### Notes
   Observed on `upstream/master` on April 15, 2026.
   


-- 
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: commits-unsubscribe@pinot.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


Re: [I] Re-enable QueryCompilationTest.testAggregateJoinRemove [pinot]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #18224:
URL: https://github.com/apache/pinot/issues/18224#issuecomment-6205099753

   This issue has been closed because it has been inactive for 180 days. If you believe this issue is still relevant, please feel free to reopen it or create a new issue. Thank you!


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


Re: [I] Re-enable QueryCompilationTest.testAggregateJoinRemove [pinot]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #18224:
URL: https://github.com/apache/pinot/issues/18224#issuecomment-5672770407

   This issue has been automatically marked as stale because it has not had any activity in the last 150 days. It will be closed in 30 days if no further activity occurs. If this issue is still relevant, please leave a comment to keep it open. Thank you for your contributions!


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


Re: [I] Re-enable QueryCompilationTest.testAggregateJoinRemove [pinot]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed issue #18224: Re-enable QueryCompilationTest.testAggregateJoinRemove
URL: https://github.com/apache/pinot/issues/18224


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org