substrait-io / substrait

A cross platform way to express data transformation, relational algebra, standardized record expression and plans.

https://substrait.io

Apache License 2.0

1.18k stars 154 forks source link

feat: clarify the meaning of plans #616

Closed westonpace closed 2 months ago

westonpace commented 6 months ago

612 and #613 highlighted a number of ambiguities in plan interpretation. I have made an attempt to clarify these based on my interpretation.

jacques-n commented 2 months ago

This looks really good. Seems like a few small edits proposed that look good and then we should be able to merge. Ping @westonpace to push over the finishline.

westonpace commented 2 months ago

Addressed comments. @jacques-n