Skip to content

[core] Support configurable Iceberg metadata path independent of storage type#5471

Merged
JingsongLi merged 2 commits intoapache:masterfrom
jahangir-yelp:iceberg_metadata_path_configurable_with_correct_database_location
Apr 15, 2025
Merged

[core] Support configurable Iceberg metadata path independent of storage type#5471
JingsongLi merged 2 commits intoapache:masterfrom
jahangir-yelp:iceberg_metadata_path_configurable_with_correct_database_location

Conversation

@jahangir-yelp
Copy link
Copy Markdown
Contributor

[core] Support configurable Iceberg metadata path independent of storage type

Purpose

Linked issue [Feature]: close #5470

This PR introduces a new table option metadata.iceberg.storage-location to allow users to configure where Iceberg metadata is stored, independently of the selected metadata.iceberg.storage type.

By default, when using metadata.iceberg.storage = table-location, Paimon does not register the table or database in Hive or AWS Glue catalogs. This is expected behavior, but it limits integration with catalog-based systems such as Lake Formation or Iceberg readers that rely on Hive/Glue.

This feature adds support for catalog registration even when using table-location, by decoupling the metadata storage path from the catalog integration logic.

This enables users to:

  • Store Iceberg metadata in the table directory
  • Still register tables and databases in Hive or Glue
  • Support hybrid deployments where metadata is colocated with table data but still discoverable via catalogs

Tests

  • Added unit tests in IcebergCommitCallbackTest
  • Manually tested using Kafka to Paimon connector with Flink and AWS Glue

API and Format

No changes to public API or storage format

Documentation

Introduces a new table option: metadata.iceberg.storage-location
Updated iceberg-compatibility.md to document the new configuration and usage scenarios

Copy link
Copy Markdown
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

+1

@JingsongLi
Copy link
Copy Markdown
Contributor

Thanks @jahangir-yelp for the contribution!

@JingsongLi JingsongLi merged commit c0e4d31 into apache:master Apr 15, 2025
21 checks passed
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.

[Feature] Support configurable Iceberg metadata path independent of storage type

2 participants