-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Re-enable QueryCompilationTest.testAggregateJoinRemove #18224
Copy link
Copy link
Open
Labels
javaPull requests that update Java codePull requests that update Java codemulti-stageRelated to the multi-stage query engineRelated to the multi-stage query enginepriority: lowNice to have, can waitNice to have, can waitqueryRelated to query processingRelated to query processingtestingRelated to tests or test infrastructureRelated to tests or test infrastructure
Metadata
Metadata
Assignees
Labels
javaPull requests that update Java codePull requests that update Java codemulti-stageRelated to the multi-stage query engineRelated to the multi-stage query enginepriority: lowNice to have, can waitNice to have, can waitqueryRelated to query processingRelated to query processingtestingRelated to tests or test infrastructureRelated to tests or test infrastructure
Type
Fields
Give feedbackNo fields configured for issues without a type.
Motivation
pinot-query-planner/src/test/java/org/apache/pinot/query/QueryCompilationTest.javastill has@IgnoreontestAggregateJoinRemove(). The ignore message says the test only passes whenPRUNE_RULESrun beforeBASIC_RULES, but enabling that changes roughly 50 hard-coded plans and may introduce redundantProjectandExchangenodes.Scope
Notes
Observed on
upstream/masteron April 15, 2026.