jtuglu1 commented on PR #19643: URL: https://github.com/apache/druid/pull/19643#issuecomment-4858637114
Drive-by comment: if we can, I'd like to expose this column extraction logic within Druid itself. A few reasons for this: 1. Other extensions/use-cases can benefit from extracting the queried columns from the plan (this can be useful in column statistics plan optimization, other emitters, etc.). 2. Having one implementation for extracting this query information ensures that our OL emitter is never out of sync with OSS parser/planner. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
