-
Notifications
You must be signed in to change notification settings - Fork 42
Dynamodb #105
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
Dynamodb #105
Conversation
|
Can one of the admins verify this patch? |
|
@brenopessoa please, update the header to 2018 |
|
@brenopessoa nice work, thank you for the help |
|
ok to test |
|
test this please |
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
Signed-off-by: Breno Pessoa <brenopessoamelo@gmail.com>
|
Uupdate the header to 2018 done , |
|
what's the status here? it would be very cool to have support for dynamodb |
|
@ulich Wanna help? |
|
yeah, but whats the status? :) without having looked at the diff, it sounds like its all done and someone just needs to merge |
|
Sure I'll review shortly. |
|
Thank you |
|
thank's for the merge. |
|
Ok, I'll take a look also :) |
|
@brenopessoa could you take a look? |
|
Hey @brenopessoa I'll just remove from the parent, Also it will be amazing if you update it to use the newest API: |
|
Yes , i'll can look it. |
|
btw, I commented the dynamodb to make green at the build: Please, don't forget to return it. |
|
@brenopessoa @otaviojava any update ? I can work on it if you need help. |
|
@amoscatelli Do you have a valid ECA in place? |
|
Hi again guys. Atm I can't handle this but if nobody else will I will make an attempt later. And the KeyValue Driver doesn't support polymorphism. For this, I believe, we need JsonB to fully support polymorphism and we'll have to wait for the current refactor that will be ready for Jakarta 10 both in spec and RI (they told me so). |
|
@otaviojava I am going to work on the Document driver soon |
|
@amoscatelli |
Hello, I tried implemented this driver, is missing the document support ,
but will start work on it as soon as I can.
Any kind of feedback are welcome. Thanks
Adds support to DynamoDB #21