paleolimbot commented on code in PR #41765:
URL: https://github.com/apache/arrow/pull/41765#discussion_r2014715189
##########
cpp/src/parquet/properties.h:
##########
@@ -1003,9 +1004,21 @@ 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 unknown logical types as their underlying physical type
+ ///
+ /// When enabled, the Arrow reader will use the underlying physical type
+ /// of a logical type that it does not recognize (e.g., one that was added
+ /// to the spec but not implemented in Parquet C++).
+ void set_convert_unknown_logical_types(bool convert_unknown_logical_types) {
Review Comment:
It seemed like a good idea to me so I renamed them all to `undefined` (easy
to rename them back if anybody feels differently)
--
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
