gianm commented on PR #14124: URL: https://github.com/apache/druid/pull/14124#issuecomment-1608263053
In talking offline with @clintropolis, I learned he's planning on doing some work to make it faster to get required bindings in the future, minimizing the need to introduce `AnalyzedExpr` throughout the code base. So, I'm thinking of reworking this patch to be limited to `Expr` caching in `PlannerContext` during SQL planning. Marking it as a draft til then. -- 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...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
