Skip to content

[HUDI-5520] Limit MDT deltacommits when data table has pending action#8772

Merged
xushiyan merged 2 commits intoapache:masterfrom
xushiyan:HUDI-5520-limit-mdt-dc
May 21, 2023
Merged

[HUDI-5520] Limit MDT deltacommits when data table has pending action#8772
xushiyan merged 2 commits intoapache:masterfrom
xushiyan:HUDI-5520-limit-mdt-dc

Conversation

@xushiyan
Copy link
Member

Change Logs

Prevent MDT deltacommits from growing infinitely when there is pending action in the data table.

New advanced config: hoodie.metadata.max.deltacommits.when_pending

Impact

MDT compaction flow.

Risk level

Low.

Documentation Update

NA

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@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

Copy link
Contributor

@zhangyue19921010 zhangyue19921010 left a comment

Choose a reason for hiding this comment

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

LGTM. just a nit comment.

if (numDeltaCommits > maxNumDeltaCommitsWhenPending) {
throw new HoodieMetadataException(String.format("Metadata table's deltacommits exceeded %d: "
+ "this is likely caused by a pending instant in the data table. Resolve the pending instant "
+ "or adjust `%s`, the restart the pipeline.",
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: the restart the pipeline. => then restart the pipeline

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks for pointing out. will fix and then land.

@xushiyan xushiyan merged commit 35aba7c into apache:master May 21, 2023
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.

3 participants