Skip to content

Retain Delta column metadata after schema changes#13418

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:delta/preserve-column-level-properties
Aug 3, 2022
Merged

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

Conversation

@alexjo2144
Copy link
Copy Markdown
Member

@alexjo2144 alexjo2144 commented Jul 29, 2022

Description

Copy over Delta Lake column metadata like nullability and invariant settings when doing schema changes.

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 Lake connector

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

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 column metadata after schema changes

@cla-bot cla-bot Bot added the cla-signed label Jul 29, 2022
@alexjo2144 alexjo2144 force-pushed the delta/preserve-column-level-properties branch from d805a8f to e88138b Compare August 1, 2022 15:09
@alexjo2144 alexjo2144 force-pushed the delta/preserve-column-level-properties branch from e88138b to 34571b2 Compare August 2, 2022 15:40
@findepi findepi requested review from ebyhr and removed request for findepi August 2, 2022 18:30
@ebyhr ebyhr merged commit 44555a0 into trinodb:master Aug 3, 2022
@ebyhr
Copy link
Copy Markdown
Member

ebyhr commented Aug 3, 2022

Merged, thanks!

@github-actions github-actions Bot added this to the 392 milestone Aug 3, 2022
@ebyhr ebyhr mentioned this pull request Aug 3, 2022
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