Skip to content

Conversation

@amoscatelli
Copy link
Contributor

@amoscatelli amoscatelli commented Jan 28, 2021

We were creating the table without waiting for it to be ready.
This caused the table not to be ready and a Resource not found Exception was thrown from aws.
This happens only when using in cloud dynamodb table, I could not reproduce the issue when using the local dynamodb docker instance.

Also adding myself as contributor where I previously contributed.

adding myself as contributor

Signed-off-by: Alessandro Moscatelli <alessandro.moscatelli@visiontech.cloud>
@genie-jnosql
Copy link
Collaborator

Can one of the admins verify this patch?

@keilw keilw requested a review from otaviojava January 28, 2021 14:28
@keilw
Copy link
Contributor

keilw commented Jan 28, 2021

@otaviojava Could you verify that please. It seems Github no longer shows merged PRs under the "contributors" in each repository or is this the first PR by @amoscatelli?

@otaviojava otaviojava merged commit 6411d96 into eclipse-jnosql:master Jan 28, 2021
@otaviojava
Copy link
Contributor

@amoscatelli thank you for the contribuition

@keilw
Copy link
Contributor

keilw commented Jan 28, 2021

Btw not sure, if his membership would allow that at this point, but would you also consider @amoscatelli for a committer election to JNoSQL or is it too soon for that?

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.

4 participants