andygrove commented on pull request #8034:
URL: https://github.com/apache/arrow/pull/8034#issuecomment-679426900
Thanks for the reviews @alamb and @jorgecarleitao .. I have implemented the
suggested and also improved the error handling so we return `Err` results
rather than panic if the
andygrove commented on pull request #8034:
URL: https://github.com/apache/arrow/pull/8034#issuecomment-678809178
> It looks fantastic! Super excited to see this!
>
> Gave it a quick look, and my understanding so far:
>
> * Physical nodes have requirements
andygrove commented on pull request #8034:
URL: https://github.com/apache/arrow/pull/8034#issuecomment-678805566
@alamb @jorgecarleitao I'm pretty excited about this PR. This is a good
example of how we can write optimizer rules against a trait-based plan.
