paleolimbot commented on code in PR #41765:
URL: https://github.com/apache/arrow/pull/41765#discussion_r2016945335
##########
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++).
+ void set_convert_undefined_logical_types(bool
convert_undefined_logical_types) {
Review Comment:
Thanks, I like `allow_` better. Updated!
--
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]
