Skip to content

feat: omit metadata columns#799

Open
parisni wants to merge 1 commit intoapache:mainfrom
leboncoin:pr-omit-hudi-mdt-column
Open

feat: omit metadata columns#799
parisni wants to merge 1 commit intoapache:mainfrom
leboncoin:pr-omit-hudi-mdt-column

Conversation

@parisni
Copy link
Contributor

@parisni parisni commented Feb 3, 2026

Important Read

  • GitHub issue: TBD

What is the purpose of the pull request

This pull request adds an option to omit Hudi metadata fields during sync, including config support and tests.

Brief change log

  • Added config flags to control omission of Hudi metadata fields.
  • Plumbed the omit option through Hudi conversion and schema extraction.
  • Added unit tests for config parsing and metadata-field filtering.

Verify this pull request

This change added tests and can be verified as follows:

Copy link
Contributor

@vinishjail97 vinishjail97 left a comment

Choose a reason for hiding this comment

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

Can we enable this config for the schema evolution test in ITConversionController ?
I have a hunch that iceberg read might fail.

if (configuration == null) {
return false;
}
String configValue = configuration.get(OMIT_METADATA_FIELDS_CONFIG);
Copy link
Contributor

Choose a reason for hiding this comment

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

We won't find OMIT_METADATA_FIELDS_CONFIG in hadoopConf..so just keep the change restricted to Properties?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@parisni
Copy link
Contributor Author

parisni commented Feb 11, 2026

Can we enable this config for the schema evolution test in ITConversionController ? I have a hunch that iceberg read might fail.

you are right the test expect the _hoodie_record_key so if we omit the column it will fail https://github.com/apache/incubator-xtable/blob/main/xtable-core/src/test/java/org/apache/xtable/ITConversionController.java#L947

Not sure what's the problem here, it's a test expectation, not a iceberg expectation

@parisni parisni requested a review from vinishjail97 February 11, 2026 15:47
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.

2 participants

Comments