Skip to content

Conversation

@rdblue
Copy link
Contributor

@rdblue rdblue commented Apr 8, 2020

Adding support for row-level deletes requires a new format version, v2. Previously, table format was checked when reading metadata, but not used anywhere else. This adds the format version to TableMetadata and updates metadata writes to pass the format version. This makes it possible to maintain v1 and iterate on v2 metadata in parallel.

@danielcweeks
Copy link
Contributor

LGTM, +1 (pending checks)

@aokolnychyi
Copy link
Contributor

Let me take a look at this tomorrow.


### Version 2

Writing metadata:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@rdblue rdblue merged commit c094da5 into apache:master Apr 9, 2020
sunchao pushed a commit to sunchao/iceberg that referenced this pull request May 10, 2023
* Internal(Boson): Move Boson iceberg code to iceberg repo apple 1.0.x-preview

* change to private static class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants