Skip to content

Document row level deletes for Iceberg#12061

Merged
findepi merged 1 commit intotrinodb:masterfrom
alexjo2144:iceberg/delete-docs
Apr 27, 2022
Merged

Document row level deletes for Iceberg#12061
findepi merged 1 commit intotrinodb:masterfrom
alexjo2144:iceberg/delete-docs

Conversation

@alexjo2144
Copy link
Member

Description

Documentation for #11886

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

Documentation of a new feature

Copy link
Member

Choose a reason for hiding this comment

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

merge-on-read in monospace looks like we support deletes only if deletion mode is merge-on-read, while we ignore it

let's not use the merge-on-read in a code block. Let's say we create deletion files.

Copy link
Member

Choose a reason for hiding this comment

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

  • for new Tables, from Trino perspective, user doesn't need to set format_version=2, because it's the default
  • for existing tables, referring to format_version may be confusing because we don't show it yet (Show Iceberg table version in SHOW CREATE TABLE #11980) and don't allow changing it (tbd)

let's just say the table must be in "v2" iceberg spec version

Copy link
Member

Choose a reason for hiding this comment

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

Is it clear that v1 and v2 refer to Iceberg spec versions?

@alexjo2144 alexjo2144 force-pushed the iceberg/delete-docs branch from 7441a4e to 23f580c Compare April 22, 2022 14:44
@alexjo2144
Copy link
Member Author

Updated, thanks

@findepi findepi merged commit 940b6d9 into trinodb:master Apr 27, 2022
@findepi findepi added the no-release-notes This pull request does not require release notes entry label Apr 27, 2022
@github-actions github-actions bot added this to the 379 milestone Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed docs no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

2 participants