Skip to content

[MINOR][DOCS] fix HiveMetastore lock provider package name#8161

Merged
danny0405 merged 1 commit intoapache:asf-sitefrom
TengHuo:asf-site
Mar 14, 2023
Merged

[MINOR][DOCS] fix HiveMetastore lock provider package name#8161
danny0405 merged 1 commit intoapache:asf-sitefrom
TengHuo:asf-site

Conversation

@TengHuo
Copy link
Contributor

@TengHuo TengHuo commented Mar 13, 2023

Change Logs

HiveMetastore lock provider full name org.apache.hudi.hive.HiveMetastoreBasedLockProvider -> org.apache.hudi.hive.transaction.lock.HiveMetastoreBasedLockProvider

According to the code https://github.com/apache/hudi/blob/master/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/transaction/lock/HiveMetastoreBasedLockProvider.java

Impact

NO impact, doc update only.

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

none

Documentation Update

Update HiveMetastore lock provider full name in concurrency control doc.

Contributor's checklist

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

Copy link
Member

@SteNicholas SteNicholas left a comment

Choose a reason for hiding this comment

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

LGTM.

@danny0405 danny0405 merged commit 64de8d0 into apache:asf-site Mar 14, 2023
@TengHuo
Copy link
Contributor Author

TengHuo commented Mar 14, 2023

Thanks guys

Zouxxyy pushed a commit to Zouxxyy/hudi that referenced this pull request Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants