-
Notifications
You must be signed in to change notification settings - Fork 28
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
The purpose of Dyno #136
Comments
A lot of the usage is identical, by design. You've hit on a few of the points here, but also missed a few:
|
So... isn't the cli something
I think this pairs well with the 'make bulk reading better' use case.
Does the AWS SDK not provide it? Is this not just a pass through? I do use this when writing tests for lambda functions that work as dynamodb triggers.
Hmmm... so another goal is to provide a clearer return state to users. Are there other cases where we should be doing this? |
Chiming in here considering the ongoing discussion in #135: There are three features of this library that make it compelling for our use:
|
I think that these two use-cases are the ones to support.
It is non-trival: see https://github.com/mapbox/dyno/blob/master/lib/serialization.js |
@rclark - why not drop this support here and move it to the replicator? |
I was talking with @rclark about the purpose of Dyno vs the javascript object friendly AWS client last night and wanted to document what we said to help us keep focused on the new goals of this project since a large part of the original goal has been solved by AWS themselves.
I'd like to rework the README to clarify these goals. Is there anything else we should cover?
I'd also like to take a look at stuff dyno does that doesn't fall under the stated goals and talk about removing the functionality or being explicit that this is a goal. One thing Dyno does now that isn't mentioned above is provide cli support.
The text was updated successfully, but these errors were encountered: