search

docker / libkv

Distributed key/value store abstraction library
Apache License 2.0
853 stars 205 forks source link

add dynamodDB #146

Open tskinn opened 7 years ago

tskinn commented 7 years ago

Signed-off-by: Taylor Skinner tskinn12@gmail.com

project0 commented 7 years ago

@tskinn are you willing to complete this PR? I would like to use the dynamodb backend as well.

tskinn commented 7 years ago

@Project0 yes I am. Is the PR missing anything for it to be merged or do we just need to bug the maintainers?

project0 commented 7 years ago

@tskinn I think fixing the CI tests will help for the approvement ;-)

tskinn commented 7 years ago

@Project0 it was the consul script that failed on that CI test run. Do you know how to trigger a run again?

project0 commented 7 years ago

@tskinn Usually changes on the branch of PR will trigger an new unit test, probably merging (rebased?) the last changes will re-run the tests.