Skip to content

[HUDI-6358] Fix flink payload merger with deletes - #8935

Merged
danny0405 merged 1 commit into
apache:masterfrom
danny0405:HUDI-6358
Jun 12, 2023
Merged

[HUDI-6358] Fix flink payload merger with deletes#8935
danny0405 merged 1 commit into
apache:masterfrom
danny0405:HUDI-6358

Conversation

@danny0405

Copy link
Copy Markdown
Contributor

Change Logs

Fixed the regression introduced by #7345.

Impact

none

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

none

Documentation Update

Describe any necessary documentation update if there is any new feature, config, or user-facing change

  • The config description must be updated if new configs are added or the default value of the configs are changed
  • Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
    ticket number here and follow the instruction to make
    changes to the website.

Contributor's checklist

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

@SteNicholas SteNicholas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@hudi-bot

Copy link
Copy Markdown
Collaborator

CI report:

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

@XuQianJin-Stars XuQianJin-Stars added the engine:flink Flink integration label Jun 12, 2023
@danny0405
danny0405 merged commit e9a67bf into apache:master Jun 12, 2023
zhangyue19921010 pushed a commit to zhangyue19921010/hudi that referenced this pull request Aug 31, 2023
…he#32)

* [HUDI-6624] Fix to return Empty when there is no commit instant (apache#9325)

* [HUDI-6358] Fix flink payload merger with deletes (apache#8935)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine:flink Flink integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants