ghuls commented on a change in pull request #9349:
URL: https://github.com/apache/arrow/pull/9349#discussion_r566271219



##########
File path: cpp/src/arrow/ipc/reader.cc
##########
@@ -58,6 +58,8 @@
 #include "generated/Schema_generated.h"
 #include "generated/SparseTensor_generated.h"
 
+#define FLATBUFFER_MAX_TABLES 10000000

Review comment:
       > According to various sources on the Internet it should be ok to just 
pass `UINT_MAX`. @wesm @emkornfield What do you think?
   
   That should be fine indeed. I got the solution for this problem from:
   https://groups.google.com/g/flatbuffers/c/JtDGnBPx9is
   
   > You simply need to pass your own max_tables to the FlatBufferBuilder 
constructor.
   > Pass e.g. INT_MAX and you will guaranteed never hit it :)
   
   I am fine with `UINT_MAX`, instead of an arbitrary value like 10000000.




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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to