Akshat-Jain commented on PR #16358: URL: https://github.com/apache/druid/pull/16358#issuecomment-2092822866
Based on offline discussion, I have modified the approach to populate a new field `Set<String> lookupsToLoad` in PlannerContext, instead of populating the `queryContext`. The new field is then used to pass info directly to MSQControllerTask when initializing it in MSQTaskQueryMaker. The primary rationale behind this change is that it allows us to limit the scope of our changes to MSQ, since queryContext is a widely used field in a lot of areas. -- 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]
