Skip to content

Conversation

@linliu-code
Copy link
Collaborator

@linliu-code linliu-code commented Oct 17, 2025

Describe the issue this Pull Request addresses

When the payload class configured during the write is different from the one from table config, the payload class validation fails the write.
When merge mode is either event_time or commit_time based, the merge behavior is not impacted by the payload class configured. Therefore, we can relax the validation constraints to ignore the payload class configuration when the merge mode is not custom.

Summary and Changelog

When the table version is >= 9, and the merge mode is not custom, we skip the payload class validation.

Tests

Unit tests for the logic change
Manual test:

  1. Using Hudi 0.14.0, generated a table and populated with OverwriteWithLatestPayload class
  2. Validated the table configurations to confirm the payload class.
  3. Using Hudi master branch, populated more data with different payload class. No error thrown.

Impact

Better user experience.

Risk Level

None.

Documentation Update

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Oct 17, 2025
@linliu-code linliu-code marked this pull request as ready for review October 17, 2025 21:30
Copy link
Contributor

@nsivabalan nsivabalan left a comment

Choose a reason for hiding this comment

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

2 minor comments.

@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@nsivabalan
Copy link
Contributor

github action is failing due to unrelated failure. going ahead w/ landing.

@nsivabalan nsivabalan merged commit a18bcd5 into apache:master Oct 19, 2025
67 of 70 checks passed
yihua pushed a commit that referenced this pull request Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants