songwdfu opened a new pull request, #16238: URL: https://github.com/apache/pinot/pull/16238
Added queryOption `usePlannerRules` that works similarly to `skipPlannerRules` #15999 . It takes the comma separated list of rules that are disabled by default that user want to enable. It only applies to defaultly disabled rules, which is defined in a static set: `CommonConstants.Broker#DEFAULT_DISABLED_RULES`. If a rule is set to be enabled and disabled by queryOption at the same time, it is disabled. Added calcite `SortJoinTransposeRule`, `SortJoinCopyRule`, and `AggregateJoinTransposeRule.EXTENDED` in `BASIC_RULES` to be disabled by default. Added tests that test these rules are disabled by default and could be enabled in `QueryPlannerRuleOptionsTest` Adjusted the skipping logic to use empty set instead of null when no options are provided to avoid NPEs. -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
