emkornfield commented on code in PR #48431:
URL: https://github.com/apache/arrow/pull/48431#discussion_r2714731978


##########
cpp/src/parquet/properties.h:
##########
@@ -139,6 +139,11 @@ class PARQUET_EXPORT ReaderProperties {
   void set_footer_read_size(size_t size) { footer_read_size_ = size; }
   size_t footer_read_size() const { return footer_read_size_; }
 
+  // If enabled, try to read the metadata3 footer from the file.
+  // If it fails, fall back to Thrift footer decoding.
+  bool read_metadata3() const { return read_metadata3_; }
+  void set_read_metadata3(bool read_metadata3) { read_metadata3_ = 
read_metadata3; }
+

Review Comment:
   I guess we need to finalize PAR2 or PAR3 footer to be able to write this out 
without extension, I think that can be follow-up work but it would be nice to 
do this as part of the FBS work to ensure we can eventually move away from 
thrift.



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