masonh22 opened a new pull request, #8028:
URL: https://github.com/apache/arrow-rs/pull/8028

   # Which issue does this PR close?
   
   - Closes #8027.
   
   # Rationale for this change
   
   This lets users detect when a Time32 or Time64 has the adjusted_to_u_t_c 
property set in the parquet file.  Previously this only worked if the file was 
written by arrow-rs and the reader chooses not to skip the arrow metadata.
   
   # What changes are included in this PR?
   
   This adds documented constant for "adjusted_to_utc" so that users are aware 
of this functionality.  This key gets added when making a `Field` from a 
parquet type if the logical type as the adjusted_to_u_t_c field set.
   
   # Are these changes tested?
   
   I updated `test_time_utc_roundtrip()` to skip the arrow metadata to simulate 
reading a file that doesn't have the metadata we expect or if a user chooses to 
skip the arrow metadata when reading.
   
   # Are there any user-facing changes?
   
   Yes: this adds a new constant "ADJUSTED_TO_UTC_KEY".  It may be helpful to 
add documentation for this near where `DataType::Time32`/`DataType::Time64` are 
defined.
   


-- 
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