Skip to content

fix: Avoid deleting metadata table with MOR during upgrade / downgrade#14191

Merged
nsivabalan merged 12 commits intoapache:masterfrom
linliu-code:issue-14171
Nov 4, 2025
Merged

fix: Avoid deleting metadata table with MOR during upgrade / downgrade#14191
nsivabalan merged 12 commits intoapache:masterfrom
linliu-code:issue-14171

Conversation

@linliu-code
Copy link
Copy Markdown
Collaborator

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

Describe the issue this Pull Request addresses

#14171

Currently while performing rollback or compaction, metadata table gets deleted since the metadata enablement configs are set to false by default. We should avoid deleting the metadata table by removing the metadata table config during compaction.

Summary and Changelog

Before we do the rollback and/or compaction, we do:

  1. Get the metadata partitions based on table config.
  2. For column_stats, partition_stats, bloom_filters, and record_index, if they are specified in table config, we set their enablement configs accordingly.
  3. Meanwhile, we check the if index.json exists. If so, we try to get the column list for column_stats, and differentiate partitioned and non-partitioned record index based on the options.

Impact

Low.

Risk Level

Low

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:M PR with lines of changes in (100, 300] label Oct 30, 2025
@yihua yihua added this to the release-1.1.0 milestone Oct 31, 2025
@github-actions github-actions bot added size:L PR with lines of changes in (300, 1000] and removed size:M PR with lines of changes in (100, 300] labels Oct 31, 2025
@linliu-code linliu-code marked this pull request as ready for review October 31, 2025 22:40
@hudi-bot
Copy link
Copy Markdown
Collaborator

hudi-bot commented Nov 3, 2025

CI report:

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

@nsivabalan nsivabalan merged commit 42066ee into apache:master Nov 4, 2025
134 of 137 checks passed
yihua added a commit to yihua/hudi that referenced this pull request Nov 10, 2025
apache#14191)

---------

Co-authored-by: Lokesh Jain <ljain@192.168.0.234>
Co-authored-by: Lokesh Jain <ljain@Lokeshs-MacBook-Pro.local>
Co-authored-by: Y Ethan Guo <yihua@apache.org>
Co-authored-by: sivabalan <n.siva.b@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-1.1.0 size:L PR with lines of changes in (300, 1000]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants