-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Don't overwrite reader and writer versions during COMMENT
and ADD COLUMN
in Delta
#14612
Conversation
@@ -263,4 +263,32 @@ public void testTrinoAlterTablePreservesTableMetadata() | |||
onTrino().executeQuery("DROP TABLE delta.default." + tableName); | |||
} | |||
} | |||
|
|||
@Test(groups = {DELTA_LAKE_DATABRICKS, DELTA_LAKE_OSS, PROFILE_SPECIFIC_TESTS}) | |||
public void testTrinoAlterTablePreservesReaderAndWriterVersions() |
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.
are we good for inserts, checkpoints, etc.?
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.
Those operations look fine. Added some statements to this test.
Can we put the minWriterVersion and minReaderVersion in the DeltaLakeTableHandle? Either way, thanks for spotting this |
72524e2
to
5ab5ab4
Compare
COMMENT
and ADD COLUMN
in DeltaCOMMENT
and ADD COLUMN
in Delta
Actually, I considered this during fixing, but we avoid adding fields to DeltaLakeTableHandle as much as possible. Let me keep as-is because the affected operations are only |
Description
Fixes #14611
Non-technical explanation
Release notes
(x) Release notes are required, with the following suggested text: