Skip to content

Improve documentation for parquet.time-zone setting#28957

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
findinpath:findinpath/docs-parquet-time-zone
Apr 7, 2026
Merged

Improve documentation for parquet.time-zone setting#28957
ebyhr merged 1 commit intotrinodb:masterfrom
findinpath:findinpath/docs-parquet-time-zone

Conversation

@findinpath
Copy link
Copy Markdown
Contributor

Description

Add more detail to hive.parquet.time-zone & delta.parquet.time-zone settings

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

## Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Apr 1, 2026
@github-actions github-actions bot added the docs label Apr 1, 2026
- `false`
* - `delta.parquet.time-zone`
- Time zone for Parquet read and write.
- Time zone used when reading timestamps from Parquet files.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

writing happens only with int96 which does not apply for Delta Lake

io.trino.parquet.writer.valuewriter.Int96TimestampValueWriter

- `true`
* - `hive.parquet.time-zone`
- Time zone for Parquet read and write.
- Time zone used when reading and writing timestamps into Parquet files.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

timestamps are being read from variant as well - i'm hopeful that the statement is generic enough

io.trino.parquet.reader.ParquetReader#readVariantAsJson

@ebyhr ebyhr merged commit 0adfd43 into trinodb:master Apr 7, 2026
18 checks passed
@github-actions github-actions bot added this to the 481 milestone Apr 7, 2026
@ebyhr ebyhr mentioned this pull request Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants