westonpace commented on PR #14082: URL: https://github.com/apache/arrow/pull/14082#issuecomment-1379501017
There is another option that some Substrait tools (validator, ibis-substrait) have used which is to re-namespace everything but that causes problems when using google.protobuf.Any because the namespace is part of the message. I think ibis-substrait works around this by somehow but it sounded complicated and I'd rather avoid it. > Split out all protobuf code in Arrow into a single shared object that other libraries depend on. (I think this is the generally recommended path, but will complicate our CMake.) We already sort of do this with Substrait, from what I see, so we'd 'just' have to share this with Flight and Flight SQL. Will this work if, sometime down the road, we end up splitting pyarrow into different wheels? I feel like there was discussion of this recently. > See if we can enable LITE_RUNTIME for one or both Protobufs. (This requires editing the Protobuf definitions.) This apparently reduces the amount of global shenanigans that Protobuf does but also disables reflection and doesn't seem to guarantee that it will fix the solution. I'm curious how this works. I'll try and read more on it. -- 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]
