[ 
https://issues.apache.org/jira/browse/ARROW-16860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17559202#comment-17559202
 ] 

Weston Pace commented on ARROW-16860:
-------------------------------------

I think I answered this offline but, for clarity (and I think this is what you 
are arguing for):

 * Start with an empty ExtensionSet
 * Convert the relations to protobuf
   * During the various ToProto calls there will be calls made to 
extension_set->EncodeFunction which will populate the extension set.
 * Once all top-level relations are converted to protobuf the extension set 
should be populated.  So now it is just a matter of converting the populated 
extension set to protobuf and adding it to the plan

> [C++] Add Roundtrip Support to Plans
> ------------------------------------
>
>                 Key: ARROW-16860
>                 URL: https://issues.apache.org/jira/browse/ARROW-16860
>             Project: Apache Arrow
>          Issue Type: Improvement
>            Reporter: Vibhatha Lakmal Abeykoon
>            Assignee: Vibhatha Lakmal Abeykoon
>            Priority: Major
>
> Convert an ACERO plan to a Substrait plan. The missing component is to add 
> `ToProto` function to convert an ACERO `ExecPlan` to a Substrait-Plan.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to