Skip to content

chore(iceberg): add @Override to registerTable overload in IcebergCatalog - #4078

Merged
adutra merged 1 commit into
apache:feature/iceberg-1.11from
sririshindra:feature/iceberg-1.11-registertableoverride
Apr 1, 2026
Merged

chore(iceberg): add @Override to registerTable overload in IcebergCatalog#4078
adutra merged 1 commit into
apache:feature/iceberg-1.11from
sririshindra:feature/iceberg-1.11-registertableoverride

Conversation

@sririshindra

Copy link
Copy Markdown
Contributor

apache/iceberg#15525 add an overloaded register table method in the Catalog.java file.
This adds the Override flag to the overloaded registerTable method.

Checklist

  • [-] 🛡️ Don't disclose security issues! (contact security@apache.org)
  • [-] 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic
  • 🧾 Updated CHANGELOG.md (if needed)
  • 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

@sririshindra
sririshindra force-pushed the feature/iceberg-1.11-registertableoverride branch from 396dd40 to 127f35c Compare March 31, 2026 19:55
@sririshindra
sririshindra marked this pull request as ready for review March 31, 2026 20:38
@sririshindra

Copy link
Copy Markdown
Contributor Author

@adutra , Could you review this PR now. Earlier the build was failing because the Iceberg snapshots were not updated. They checks are passing now after the snapshots are updated.

@github-project-automation github-project-automation Bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Mar 31, 2026
@adutra
adutra merged commit 3a31410 into apache:feature/iceberg-1.11 Apr 1, 2026
17 checks passed
@github-project-automation github-project-automation Bot moved this from Ready to merge to Done in Basic Kanban Board Apr 1, 2026
@adutra

adutra commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Thanks @sririshindra !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants