isidentical commented on PR #3912: URL: https://github.com/apache/arrow-datafusion/pull/3912#issuecomment-1306340705
@alamb I think that sounds like a good plan! The primary place where this would be essential (or at least, I'd interpret it as essential; but maybe we can find a simpler solution) is `AND`s which we can work on in the subsequent PRs. For this PR, I've dropped support for 'intermediate column updates': so `apply()` API is gone, `context` is now shared as a regular reference (`&AnalysisContext`) and all the phases for intermediate analyses are removed (e.g. new upper bound computation from comparison analyzer). -- 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]
