Skip to content

[iceberg] Support TagCallback.notifyCreation without snapshot id#7772

Open
ArnavBalyan wants to merge 1 commit intoapache:masterfrom
ArnavBalyan:arnav/fix-tag
Open

[iceberg] Support TagCallback.notifyCreation without snapshot id#7772
ArnavBalyan wants to merge 1 commit intoapache:masterfrom
ArnavBalyan:arnav/fix-tag

Conversation

@ArnavBalyan
Copy link
Copy Markdown
Member

Purpose

  • notifyCreation(String) throws UnsupportedOperationException breaking the TagCallback contract.
  • The snapshot id is available to the iceberg compat layer, TagManager persists the tag file before invoking a callback.
  • Ensure we can support Tagmanager notify creation based on the tag id.

Tests

  • UT

@ArnavBalyan
Copy link
Copy Markdown
Member Author

cc @JingsongLi thanks :)

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.

1 participant