aykut-bozkurt commented on PR #6299:
URL: https://github.com/apache/arrow-rs/pull/6299#issuecomment-2308413887

   > This appears to be writing nanoseconds as milliseconds, which is incorrect?
   
   Physically it will not lose precision since we write it only if it <= 
i32::MAX. But yeah, logically seems weird since we write a `IntervalNano` value 
to parquet's milliseconds part.
   
   I think it could be an option to check a metadata key like 
`adjusted_to_milliseconds` or something like that. In that way, writer would 
know what they do.
   
   There is currently no way to write the interval with millis to parquet via 
arrow. And it seems that we can get rid of the current limitation.


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

Reply via email to