Skip to content

[HUDI-7048] Fix checkpoint loss issue when changing MOR to COW in streamer#10001

Merged
danny0405 merged 1 commit intoapache:masterfrom
waitingF:fix-checkpoint-loss-issue-when-changing-mor-to-cow-in-streamer
Nov 8, 2023
Merged

[HUDI-7048] Fix checkpoint loss issue when changing MOR to COW in streamer#10001
danny0405 merged 1 commit intoapache:masterfrom
waitingF:fix-checkpoint-loss-issue-when-changing-mor-to-cow-in-streamer

Conversation

@waitingF
Copy link
Contributor

@waitingF waitingF commented Nov 7, 2023

Change Logs

Before changing MOR to COW, we will do a full compaction. But in the .commit meta file generated by compaction, there is no checkpoint info. So in such table type changing case, there is dataloss issue.

The pr is going to fix this.

Impact

none

Risk level (write none, low medium or high below)

low

Documentation Update

none

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

hudi-bot commented Nov 7, 2023

CI report:

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

@danny0405 danny0405 added engine:spark Spark integration area:ingest Ingestion into Hudi labels Nov 8, 2023
@danny0405 danny0405 changed the title [MINOR] fix checkpoint loss issue when changing MOR to COW in streamer [HUDI-7048] Fix checkpoint loss issue when changing MOR to COW in streamer Nov 8, 2023
@danny0405 danny0405 merged commit eeec775 into apache:master Nov 8, 2023
nsivabalan pushed a commit to nsivabalan/hudi that referenced this pull request Nov 23, 2023
…eamer (apache#10001)

Co-authored-by: wei.kong <wei.kong@shopee.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ingest Ingestion into Hudi engine:spark Spark integration

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants