Jackie-Jiang opened a new pull request #6824:
URL: https://github.com/apache/incubator-pinot/pull/6824


   ## Description
   Currently in `BaseBrokerRequestHandler`, all the query rewrite and 
optimization applies to both PQL `BrokerRequest` and SQL `PinotQuery`, which is 
doubling the work.
   In this PR, we separate the SQL and PQL query handling, and only process one 
of `BrokerRequest` and `PinotQuery`.
   Also modify the `PartitionSegmentPruner` and `TimeSegmentPruner` to support 
segment pruning based on SQL `PinotQuery`.
   
   There are some duplicate code in `BaseBrokerRequestHandler` for SQL and PQL 
query handling, which is intentional. The PQL query handling code is subject to 
be removed in the future.


-- 
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:
us...@infra.apache.org



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

Reply via email to