Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HUDI-3147] Add endpoint_url to dynamodb lock provider #4500

Merged

Conversation

parisni
Copy link
Contributor

@parisni parisni commented Jan 3, 2022

What is the purpose of the pull request

This adds a config param to specify the endpoint url for dynamodb lock provider

fixes #4499

Verify this pull request

I get trouble with applying checkstyle in this guide https://hudi.apache.org/contribute/developer-setup

@hudi-bot
Copy link

hudi-bot commented Jan 3, 2022

CI report:

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

@nsivabalan nsivabalan changed the title [MINOR] Add endpoint_url to dynamodb lock provider [HUDI-3147] Add endpoint_url to dynamodb lock provider Jan 3, 2022
@vinothchandar vinothchandar added this to Ready for Review in PR Tracker Board Jan 4, 2022
@nsivabalan nsivabalan added the priority:critical production down; pipelines stalled; Need help asap. label Jan 4, 2022
Copy link
Contributor

@nsivabalan nsivabalan left a comment

Choose a reason for hiding this comment

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

LGTM

PR Tracker Board automation moved this from Ready for Review to Nearing Landing Jan 4, 2022
@nsivabalan nsivabalan merged commit 37b15ff into apache:master Jan 4, 2022
PR Tracker Board automation moved this from Nearing Landing to Done Jan 4, 2022
@vinishjail97 vinishjail97 mentioned this pull request Jan 5, 2022
5 tasks
nsivabalan pushed a commit that referenced this pull request Jan 7, 2022
Co-authored-by: Nicolas Paris <nicolas.paris@adevinta.com>
@vinishjail97 vinishjail97 mentioned this pull request Jan 24, 2022
5 tasks
vingov pushed a commit to vingov/hudi that referenced this pull request Jan 26, 2022
Co-authored-by: Nicolas Paris <nicolas.paris@adevinta.com>
liusenhua pushed a commit to liusenhua/hudi that referenced this pull request Mar 1, 2022
Co-authored-by: Nicolas Paris <nicolas.paris@adevinta.com>
vingov pushed a commit to vingov/hudi that referenced this pull request Apr 3, 2022
Co-authored-by: Nicolas Paris <nicolas.paris@adevinta.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:critical production down; pipelines stalled; Need help asap.
Projects
Development

Successfully merging this pull request may close these issues.

[SUPPORT] DynamoDBBasedLockProvider support for local dynamodb
4 participants