Skip to content

[HUDI-9673] Remove event_time metadata support from DefaultHoodieRecordPayload#13662

Merged
danny0405 merged 1 commit intoapache:masterfrom
linliu-code:HUDI-9673
Aug 1, 2025
Merged

[HUDI-9673] Remove event_time metadata support from DefaultHoodieRecordPayload#13662
danny0405 merged 1 commit intoapache:masterfrom
linliu-code:HUDI-9673

Conversation

@linliu-code
Copy link
Copy Markdown
Collaborator

Change Logs

Since event_time is populated in HoodieWriteHandle, there is no need to duplicate the logic in DefaultHoodieRecordPayload. So far, event_time is the only payload that supports event_time metadata.
Therefore, we remove the support from DefaultHoodieRecordPayload now.

Impact

No duplicated support for event_time from DefaultHoodieRecordPayload.

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

Low.

Documentation Update

Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

  • 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

@github-actions github-actions Bot added the size:S PR with lines of changes in (10, 100] label Jul 31, 2025
@linliu-code linliu-code changed the title [HUDI-9673] Remove event_time metadata from DefaultHoodieRecordPayload [HUDI-9673] Remove event_time metadata support from DefaultHoodieRecordPayload Jul 31, 2025
@linliu-code linliu-code marked this pull request as ready for review July 31, 2025 18:20
@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

@danny0405 danny0405 merged commit 2869cb7 into apache:master Aug 1, 2025
60 checks passed
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