Skip to content

[core] fix the timezone conversion for timestamp_ltz data_type in OrcFileFormat#5082

Merged
leaves12138 merged 1 commit intoapache:masterfrom
xiangyuf:orc-fix
Feb 17, 2025
Merged

[core] fix the timezone conversion for timestamp_ltz data_type in OrcFileFormat#5082
leaves12138 merged 1 commit intoapache:masterfrom
xiangyuf:orc-fix

Conversation

@xiangyuf
Copy link
Copy Markdown
Contributor

@xiangyuf xiangyuf commented Feb 13, 2025

Purpose

Linked issue: close #5066

Tests

OrcFormatReadWriteTest#testTimestampLTZWithLegacyWriteAndRead
OrcFormatReadWriteTest#testTimestampLTZWithNewWriteAndRead
OrcFormatReadWriteTest#testTimestampLTZWithNewWriteAndLegacyRead
OrcFormatReadWriteTest#testTimestampLTZWithLegacyWriteAndNewRead

API and Format

orc-format

Documentation

@xiangyuf
Copy link
Copy Markdown
Contributor Author

@JingsongLi PTAL

@xiangyuf xiangyuf closed this Feb 14, 2025
@xiangyuf xiangyuf changed the title [core] fix the timeshit error for timestamp_ltz data_type in OrcFileFormat [core] fix the wrong timezone behavior for timestamp_ltz data_type in OrcFileFormat Feb 14, 2025
@xiangyuf xiangyuf reopened this Feb 14, 2025
@xiangyuf xiangyuf changed the title [core] fix the wrong timezone behavior for timestamp_ltz data_type in OrcFileFormat [core] fix the timezone conversion for timestamp_ltz data_type in OrcFileFormat Feb 14, 2025
@xiangyuf
Copy link
Copy Markdown
Contributor Author

CI timeout. Passed in local repo:
image

@xiangyuf
Copy link
Copy Markdown
Contributor Author

@leaves12138
Copy link
Copy Markdown
Contributor

+1

@leaves12138 leaves12138 merged commit a5c9111 into apache:master Feb 17, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Parquet format with timestamp with local time processing wrong

2 participants