ryzhyk opened a new issue, #6949:
URL: https://github.com/apache/arrow-rs/issues/6949

   `convert_timestamp_millis_to_string` and 
`convert_timestamp_micros_to_string` round the timestamp down to seconds, thus 
losing any fractional seconds when converting the timestamp to JSON:
   
   
https://github.com/apache/arrow-rs/blob/4f1f6e57c568fae8233ab9da7d7c7acdaea4112a/parquet/src/record/api.rs#L926
   
   Wouldn't it be better to print the fractional seconds in the RFC3339 format 
using `%.3f` for millis and %.6f for micros, like so:
   
   ```
       format!("{}", dt.format("%Y-%m-%d %H:%M:%S%.3f %:z"))
   ```
   
   This is a simple fix, and I am happy to create a PR, but was wondering if 
there's any reason for the current approach, which throws away fractional 
seconds.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to