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 (`required_child_distribution`) > > * Physical nodes can fulfill requirements (`output_partitioning`) > > * The physical optimizer knows which physical nodes fulfill certain requirements (Merge fulfills `Distribution::SinglePartition` so far), and introduces them to fulfill a nodes' requirements, by using `children` to get the node's children and `with_new_children` to re-write the node with new children (that fulfill requirements). > > > Is this a good overview of this PR (before going into the specifics)? Yes, I think that's a good overview. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
