Skip to content

[HUDI-9385] Remove start commit with time#13330

Merged
danny0405 merged 1 commit intoapache:masterfrom
the-other-tim-brown:deprecate-start-commit-with-time-2
May 21, 2025
Merged

[HUDI-9385] Remove start commit with time#13330
danny0405 merged 1 commit intoapache:masterfrom
the-other-tim-brown:deprecate-start-commit-with-time-2

Conversation

@the-other-tim-brown
Copy link
Contributor

Change Logs

  • Remove startCommitWithTime usage outside of tests
  • Update methods so only tests can specify a specific time

Impact

  • Cleans up methods to ensure users start commit to generate a timestamp instead of supplying an arbitrary value

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. 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

@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

@github-actions github-actions bot added the size:L PR with lines of changes in (300, 1000] label May 21, 2025
Copy link
Contributor

@danny0405 danny0405 left a comment

Choose a reason for hiding this comment

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

+1, the fix the straight-forward, land it first.

@danny0405 danny0405 merged commit 652c7a3 into apache:master May 21, 2025
58 checks passed
@the-other-tim-brown the-other-tim-brown deleted the deprecate-start-commit-with-time-2 branch May 21, 2025 13:36
alexr17 pushed a commit to alexr17/hudi that referenced this pull request Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants