Jackie-Jiang opened a new pull request #6423: URL: https://github.com/apache/incubator-pinot/pull/6423
## Description Currently broker only optimizes the filter in BrokerRequest, but leaves the filter in PinotQuery as is. Because of this, we can only use the filter from the BrokerRequest, which is inefficient because there is no native Expression support. This PR replaces the BrokerRequestOptimizer with QueryOptimizer to also optimize the filter in PinotQuery, so that for SQL queries, we can use PinotQuery to represent the query. It will make deprecation of PQL much easier. This PR also removed the deprecated range predicate delimiter. For backward-compatibility, Broker will optimize the filter in both BrokerRequest and PinotQuery. ## Upgrade Notes Please upgrade Brokers before Servers ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
