walterddr opened a new issue, #10871: URL: https://github.com/apache/pinot/issues/10871
Description == when query broker for a specific query. there's no need to parse out the tenant info. however when using the Pinot controller UI. controller needs to first detect all the tables accessed by the query, then find the common tenant of a broker to route the query to. Status Quo == currently this is done via a util done in #10336. however this util has lots of drawback and overhead. b/c it is not running the actual planner. Proposed solution == 1. run the calcite planner on controller (`QueryEnvironment.compileQuery`) 2. and use `RelOptUtils.findTables` to find all tables accessed by this query. 3. use the same util currently available to find common broker tenant list. -- 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.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
