Skip to content

Conversation

@alexr17
Copy link
Contributor

@alexr17 alexr17 commented Aug 12, 2025

Change Logs

Adds the abstract implementation of the storage-based lock provider using conditional writes.

See RFC: #12927

Based on 1.x PR: #13103

Impact

Adds abstract lock provider implementation

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

None, without StorageLock implementation, this class will not be used.

Documentation Update

Config updates added here: #13104

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:XL PR with lines of changes > 1000 label Aug 12, 2025
Copy link
Contributor

@yihua yihua left a comment

Choose a reason for hiding this comment

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

LGTM

@apache apache deleted a comment from hudi-bot Aug 12, 2025
@yihua
Copy link
Contributor

yihua commented Aug 12, 2025

@hudi-bot run azure

@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

@yihua yihua merged commit 949017e into apache:branch-0.x Aug 12, 2025
48 checks passed
alexr17 added a commit to alexr17/hudi that referenced this pull request Aug 13, 2025
alexr17 added a commit to alexr17/hudi that referenced this pull request Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL PR with lines of changes > 1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants