wgtmac commented on code in PR #50916:
URL: https://github.com/apache/arrow/pull/50916#discussion_r3922300953
##########
cpp/src/parquet/properties.h:
##########
@@ -1157,7 +1157,9 @@ class PARQUET_EXPORT ArrowReaderProperties {
list_type_(kArrowDefaultListType),
arrow_extensions_enabled_(false),
should_load_statistics_(false),
- smallest_decimal_enabled_(false) {}
+ smallest_decimal_enabled_(false),
+ convert_flba_timestamps_(false),
Review Comment:
SGTM
##########
cpp/src/parquet/properties.h:
##########
@@ -1295,6 +1297,29 @@ class PARQUET_EXPORT ArrowReaderProperties {
/// this setting will be ignored.
bool smallest_decimal_enabled() const { return smallest_decimal_enabled_; }
+ /// \brief Set whether to infer Arrow timestamps from Parquet FLBA types.
+ ///
+ /// When enabled, Parquet FLBA(12) TIMESTAMP columns are read as Arrow
timestamps.
+ /// VAlues that do not fit in 64 bit timestamps are handled per
Review Comment:
```suggestion
/// Values that do not fit in 64 bit timestamps are handled per
```
--
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]
