wesm commented on pull request #10934: URL: https://github.com/apache/arrow/pull/10934#issuecomment-919626099
Per my comments on the mailing list, if this helps unblock our experimentation with a serialized-query-plan-type interface in C++, and it is abundantly clear that it is experimental (it might be a good idea to steer clear of the format/ directory altogether, somewhere else perhaps?) and aiding with dependent developments, then merging this seems okay with me. Hopefully we can work toward convergence with what is being discussed in Substrait (though having to use protobufs fills me with dread — we could use https://github.com/nanopb/nanopb to avoid taking on a libprotobuf dependency that we don't want in libarrow), but if not we can cross that bridge when we get there. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
