dongjoon-hyun commented on PR #1688: URL: https://github.com/apache/orc/pull/1688#issuecomment-1848249495
Thank you, @wgtmac and @guiyanakuang . To start and maintain `Apache ORC Format 1.0.0` in the official ProtocolBuffer way, this is the minimum required change due to the following. - https://github.com/apache/orc-format/pull/5 As we know, `ProtocolBuffers` is a safe protocol for field renaming in terms of binary compatibility. So, Apache ORC 1.x libraries (Java/C++) can read and write `ORC Format v1.0` files without any problems. However, the generated C++ code and dump meta info is different like we see in this PR. The new C++ API and dump meta data follows an official ProtocolBuffers recommendation. So, I believe we can help and persuade the down-streams like Apache Arrow easily without many blockers. -- 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]
