Skip to content

[HUDI-7728] Use StorageConfiguration in LockProvider constructors#11173

Merged
yihua merged 1 commit intoapache:masterfrom
yihua:HUDI-7728-lock-provider-storage-conf
May 8, 2024
Merged

[HUDI-7728] Use StorageConfiguration in LockProvider constructors#11173
yihua merged 1 commit intoapache:masterfrom
yihua:HUDI-7728-lock-provider-storage-conf

Conversation

@yihua
Copy link
Contributor

@yihua yihua commented May 8, 2024

Change Logs

As above, to avoid using Hadoop's Configuration class in the constructor.

Impact

Removes Configuration class usage in LockProvider to make hudi-common module Hadoop independent.

Risk level

none

Documentation Update

none

Contributor's checklist

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

@yihua yihua force-pushed the HUDI-7728-lock-provider-storage-conf branch from 8438ce2 to 1e91209 Compare May 8, 2024 01:10
@github-actions github-actions bot added the size:M PR with lines of changes in (100, 300] label May 8, 2024
@yihua yihua force-pushed the HUDI-7728-lock-provider-storage-conf branch 2 times, most recently from e3a0482 to 26ecd31 Compare May 8, 2024 02:10
@yihua yihua force-pushed the HUDI-7728-lock-provider-storage-conf branch from 26ecd31 to 2fb5aa3 Compare May 8, 2024 02:14
@apache apache deleted a comment from hudi-bot May 8, 2024
@apache apache deleted a comment from hudi-bot May 8, 2024
@hudi-bot
Copy link
Collaborator

hudi-bot commented May 8, 2024

CI report:

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M PR with lines of changes in (100, 300]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants