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

Reply via email to