Skip to content

Conversation

alexr17
Copy link
Contributor

@alexr17 alexr17 commented Aug 12, 2025

Change Logs

This is the 1st PR of the 0.x cherry-pick for the StorageBasedLockProvider - see the original: #12954. This adds an interface, StorageLockClient and various supplemental models. The main lock provider algorithm will be added in StorageBasedLockProvider however we define the contract for our pluggable storage format here to make it clear what is not being abstracted away.

Each StorageLockFile payload will be written to object storage, and we add several helper methods in this PR to ensure the calling storage client apis can serialize and deserialize the payload.

See RFC: #12927

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. If not, put "none".

None

Contributor's checklist

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

@alexr17 alexr17 changed the base branch from master to branch-0.x August 12, 2025 02:57
@github-actions github-actions bot added the size:L PR with lines of changes in (300, 1000] label Aug 12, 2025
@hudi-bot
Copy link

CI report:

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

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

@yihua
Copy link
Contributor

yihua commented Aug 12, 2025

CI failures are irrelevant. Merging the PR.

@yihua yihua merged commit 00bf6a3 into apache:branch-0.x Aug 12, 2025
89 of 95 checks passed
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