pitrou commented on PR #38054:
URL: https://github.com/apache/arrow/pull/38054#issuecomment-1752636459

   Regardless of which flatc version is being used, it seems we should perhaps 
upgrade the [vendored flatbuffers 
headers](https://github.com/apache/arrow/tree/main/cpp/thirdparty/flatbuffers) 
to the same version.
   
   For example, flatc 23.5.26 generates the following version check which seems 
to imply that the versions should match:
   ```c++
   // Ensure the included flatbuffers.h is the same version as when this file 
was
   // generated, otherwise it may not be compatible.
   static_assert(FLATBUFFERS_VERSION_MAJOR == 23 &&
                 FLATBUFFERS_VERSION_MINOR == 5 &&
                 FLATBUFFERS_VERSION_REVISION == 26,
                "Non-compatible flatbuffers version included");
   ```


-- 
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]

Reply via email to