jorisvandenbossche commented on issue #39489: URL: https://github.com/apache/arrow/issues/39489#issuecomment-1884471274
> Hmmm this regard `isAdjustedToUTC=true`, however, the col is `col: timestamp[us]`. So previously I think it's a bug :-( As I tried to explain: no, it's not a bug, it was an explicit design decision (the links I posted above, https://github.com/apache/arrow/issues/22303 / https://github.com/apache/arrow/pull/4856). When creating the ParquetSchema from parsing the Parquet file/thrift, we did set `isAdjustedToUTC=true`, but we also added a `is_from_converted_type=true` (if it was created from a legacy file with ConvertedTypes). So the ParquetSchema follows the Parquet spec, but the on conversion of this ParsuetSchema to an ArrowSchema, we explicitly had logic to ignore `isAdjustedToUTC` when ``is_from_converted_type=true`` (that's the line of code you changed in the PR). -- 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
