Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions docs/src/main/sphinx/object-storage/file-formats.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,14 @@ with ORC files performed by supported object storage connectors:
[](file-compression) is automatically performed and some details can be
configured.

## ORC support limitations

[Trino ignores Calendar entry in ORC file metadata.](https://github.com/trinodb/trino/issues/26865)
As a result Trino always treats dates and timestamps as values written using
proleptic Gregorian calendar. This causes incorrect values read when reading
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
proleptic Gregorian calendar. This causes incorrect values read when reading
Gregorian calendar. This causes incorrect values when reading

Copy link
Member

Choose a reason for hiding this comment

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

  • proleptic is an important word, we should keep it

  • "This causes incorrect values when reading" reads as missing something.
    Incorrect values where? does the file gets broken, changed or malformed when we try to read it? or only the values reconstructed in computer memory are wrong?

date/time values before Oct 15, 1582 that were written using hybrid
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
date/time values before Oct 15, 1582 that were written using hybrid
values before Oct 15, 1582 that were written using hybrid

Copy link
Member

Choose a reason for hiding this comment

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

we can remove "date/time", but then we need to indicate which types does this apply to.
@mosabua how would you want to indicate this?

Julian-Gregorian calendar.

(parquet-format-configuration)=
## Parquet format configuration properties

Expand Down