masonh22 commented on PR #8028: URL: https://github.com/apache/arrow-rs/pull/8028#issuecomment-3160648451
> I think I am concerned that we will be adding additional overhead to all readers of schemas even when the feature is not widely used I agree, I wish there was a better way to do this. I'm guessing that adding a "zoned" boolean field to Time32 and Time64 is out of the question? Anyways, aside from the check for whether a field is a zoned Time type, this only adds overhead if the parquet file is using zoned timestamps. For "pure" arrow-rs users who aren't reading files from other writers, this will never come up. I'm fine with sitting on this for a bit if you or others want to take more time to consider this. This isn't a huge priority for me. -- 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
