pitrou commented on code in PR #41765: URL: https://github.com/apache/arrow/pull/41765#discussion_r2016187926
########## cpp/src/parquet/properties.h: ########## @@ -1003,9 +1004,23 @@ class PARQUET_EXPORT ArrowReaderProperties { void set_should_load_statistics(bool should_load_statistics) { should_load_statistics_ = should_load_statistics; } + /// Return whether loading statistics as much as possible. bool should_load_statistics() const { return should_load_statistics_; } + /// Convert undefined logical types as their underlying physical type + /// + /// When enabled, the Arrow reader will use the underlying physical type to deduce the + /// Arrow type when the logical type is not recognized (e.g., one that was added to the + /// spec but not implemented in Parquet C++). Review Comment: Also say what happens if this option is false. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org