-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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.
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
Add support for iceberg set properties #21606
Add support for iceberg set properties #21606
Conversation
Cherry-pick of trinodb/trino@72df2b9 Co-authored-by: ebyhr
Cherry-pick of trinodb/trino@cf368de Co-authored-by: ebyhr
Codenotify: Notifying subscribers in CODENOTIFY files for diff 0f89633...ba42733.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the doc! A couple of small suggestions - if they don't look good to you, let me know and we'll figure it out.
Thank you for your review @steveburnett . I am closing the PR and these changes are now added to #21495 as a separate commit. I have addressed your review comments there itself. |
Description
Add support for Iceberg ALTER TABLE SET PROPERTIES
Needs to merge after #21495 and #21250.
Docs and testing is pending.
Motivation and Context
#21251
Impact
ALTER TABLE foo SET PROPERTIES (commit_retries = 5);
Test Plan
Unit testing (pending) and manual testing (done)
Contributor checklist
Release Notes
Please follow release notes guidelines and fill in the release notes below.