gortiz opened a new pull request, #15538: URL: https://github.com/apache/pinot/pull/15538
Some weeks ago, I modified QueryEnvironment to include a new class called CompiledQuery. This class contains the result of parsing, validating, transforming into RelNode and then optimizing the query. Today, I'm splitting this process into some extra steps: validation, to relation and optimization. These steps were already there, but calling them individually was impossible. This isn't ideal since we may want to stop the process in the middle or continue after applying some changes. The clearest case is PinotQueryResource.getTableNames(), which needed to optimize the query just to get the used tables. That optimization process there is not necessary (at least assuming queries are not written in such a way that whole tables can be statically pruned), and in this PR, we are using the relational model without optimizations to list the tables. This should accelerate queries when executed from the controller instead of directly attacking the brokers. -- 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
