Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Backport https://github.com/prestodb/presto/pull/22926 into 0.285, 0.286 and 0.287 #23011

Open
wanglinsong opened this issue Jun 14, 2024 · 4 comments
Open

Backport https://github.com/prestodb/presto/pull/22926 into 0.285, 0.286 and 0.287 #23011

wanglinsong opened this issue Jun 14, 2024 · 4 comments
Assignees
@wanglinsong

Comments

@wanglinsong
Copy link
Member

wanglinsong commented Jun 14, 2024 •
edited
Loading

#22926
CC @tdcmeehan
@wanglinsong wanglinsong self-assigned this Jun 14, 2024
@wanglinsong
Copy link
Member Author

@tdcmeehan which release to patch first?

@tdcmeehan
Copy link
Contributor

We can start backwards from 0.285

@wanglinsong
Copy link
Member Author

➜ presto git:(release-0.285) ✗ git cherry-pick 9bce03a

@wanglinsong
Copy link
Member Author

wanglinsong commented Jun 28, 2024 •
edited
Loading 

➜  presto git:(release-0.285) ✗ git cherry-pick 9bce03a8d2804a40741390aae4d5107a0c08f5dc

Auto-merging presto-iceberg/src/main/java/com/facebook/presto/iceberg/IcebergAbstractMetadata.java
CONFLICT (content): Merge conflict in presto-iceberg/src/main/java/com/facebook/presto/iceberg/IcebergAbstractMetadata.java
Auto-merging presto-iceberg/src/test/java/com/facebook/presto/iceberg/IcebergDistributedSmokeTestBase.java
error: could not apply 9bce03a8d2... Disable timestamp with time zone type on create & alter table for iceberg tables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wanglinsong wanglinsong

Labels
None yet
Projects
Build, releases and testing
Status: 📋 Prioritized Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tdcmeehan @wanglinsong