gortiz commented on PR #14193: URL: https://github.com/apache/pinot/pull/14193#issuecomment-2443946252
My suggestion is to modify the parser in order to support our different modes. It would be something like: - `EXPLAIN PLAN SEGMENT [INFO] FOR` _always_ shows the plan asking segments. - `EXPLAIN PLAN WITH LOGICAL [INFO] FOR` _always_ shows the logical plan. - `EXPLAIN PLAN WITH WORKER [INFO] FOR` _always_ shows the plan with workers. - `EXPLAIN PLAN FOR` shows the plan asking segments or the logical plan depending on the configuration. Or some other syntax like `EXPLAIN SEGMENT PLAN`, `EXPLAIN WORKERS PLAN`, `EXPLAIN LOGICAL PLAN`, etc. The current `EXPLAIN PLAN WITH/WITHOUT IMPLEMENTATION` and `EXPLAIN IMPLEMENTATION PLAN` should be still supported but we can either not document them or recommend to not use them -- 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
