westonpace commented on PR #13401: URL: https://github.com/apache/arrow/pull/13401#issuecomment-1208677326
> Since this API is designed to be a test-only utility I wouldn't expect us to need to deal with custom exec nodes @bkietz That's a reasonable stance. I suppose I had hoped we could provide some utility to users with custom relations. I suppose the internal-ness of Substrait makes that pretty difficult however. Custom relations in Substrait use the `any` message type in protobuf. Is it legal to expose `google/protobuf/any.pb.h`? If yes, then it seems we could still have a registry (or at least a fallback lambda) for custom extensions. If no, then won't this be a concern for the other direction, e.g. deserializing Substrait plans with custom extensions? -- 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]
