Skip to content

Retain Delta metadata properties after schema changes#13368

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:delta/preserve-metadata-properties
Jul 27, 2022
Merged

Retain Delta metadata properties after schema changes#13368
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:delta/preserve-metadata-properties

Conversation

@alexjo2144
Copy link
Copy Markdown
Member

Description

Schema changes like add column, setting column comment, and setting
table comment inadvertently cleared the MetadataEntry configuration
field.

Is this change a fix, improvement, new feature, refactoring, or other?

Fix

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Delta connector

How would you describe this change to a non-technical end user or system administrator?

Ensure Delta connector operations do not reset table properties

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
(x) Release notes entries required with the following suggested text:

# Delta Lake
* Retain Delta metadata properties after schema changes

Schema changes like add column, setting column comment, and setting
table comment inadvertently cleared the MetadataEntry configuration
field.
@alexjo2144 alexjo2144 force-pushed the delta/preserve-metadata-properties branch from d315de7 to a018bc7 Compare July 27, 2022 14:08
@alexjo2144 alexjo2144 requested a review from ebyhr July 27, 2022 14:49
@ebyhr ebyhr merged commit fa559d0 into trinodb:master Jul 27, 2022
@ebyhr
Copy link
Copy Markdown
Member

ebyhr commented Jul 27, 2022

Merged, thanks!

@github-actions github-actions bot added this to the 392 milestone Jul 27, 2022
@ebyhr ebyhr mentioned this pull request Jul 27, 2022
@alexjo2144 alexjo2144 deleted the delta/preserve-metadata-properties branch July 28, 2022 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants