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

Allow DynamoDB table name to be specified #61

Closed
wants to merge 1 commit into from

Conversation

samcday
Copy link

@samcday samcday commented Mar 20, 2016

No description provided.

@dlsniper
Copy link

@samcday would you like to update the PR to fix the conflicts? If not, I can open a separate one but I'd rather have this one merged than have two different PRs solving the same thing.

@samcday samcday force-pushed the custom-table-name branch from 0498b25 to 85962d7 Compare March 29, 2016 18:47
@samcday
Copy link
Author

samcday commented Mar 29, 2016

@dlsniper sorted! Pesky formatting changes in upstream ;)

EDIT: actually, more like pesky auto-formatting in my IDEA. Derp.

@pfifer
Copy link
Contributor

pfifer commented Jul 7, 2016

I've started the process of merging the pull request internally. Unfortunately I had to delay merging this for the 1.6.4 release. I do plan to have this merged, and released in the near future.

@pfifer
Copy link
Contributor

pfifer commented Jul 11, 2016

Please confirm that we can use, modify, copy, and redistribute this contribution. After I get the confirmations I can begin the release process.

Thanks.

@samcday
Copy link
Author

samcday commented Jul 11, 2016

Confirmed.

pfifer added a commit that referenced this pull request Jul 21, 2016
Merge, and slight adjustment, of
#61.

This allows configuration to explicitly set the name of the lease
table.  The table name defaults to the application name which is the
current behavior.
@pfifer pfifer added this to the v1.6.5 milestone Jul 21, 2016
@pfifer
Copy link
Contributor

pfifer commented Jul 25, 2016

This change is committed, and will be released to Maven soon.

@pfifer pfifer closed this Jul 25, 2016
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