kangakum36 commented on code in PR #43268:
URL: https://github.com/apache/arrow/pull/43268#discussion_r1679853945
##########
cpp/src/parquet/arrow/schema.cc:
##########
@@ -382,18 +382,18 @@ Status FieldToNode(const std::string& name, const
std::shared_ptr<Field>& field,
break;
case ArrowTypeId::TIME32:
type = ParquetType::INT32;
- logical_type =
- LogicalType::Time(/*is_adjusted_to_utc=*/true,
LogicalType::TimeUnit::MILLIS);
+ logical_type =
LogicalType::Time(arrow_properties.time_is_adjusted_to_utc(),
Review Comment:
It might also be worth noting that a user can use `pq.ParquetFile.schema` in
python or use the `ParquetFileReader`'s metadata in c++ to get the
`is_adjusted_to_utc` field from the parquet schema (even though its not exposed
in the arrow schema or TIME32 or TIME64 arrow types)
--
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]
