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

State of the Dynamodb package #16

Closed
lostghost opened this issue Aug 30, 2013 · 2 comments
Closed

State of the Dynamodb package #16

lostghost opened this issue Aug 30, 2013 · 2 comments

Comments

@lostghost
Copy link
Contributor

How willing are you to accept aggressive changes to the dynamodb library that change the exposed interface? Obviously changing or removing exported types and functions would impact anyone currently using this package, but the current state of this library feels pretty incomplete and like it should probably be in the /exp/ directory at the moment.

Before I make major changes to the library and submit pull requests, how willing are you to break from the current interface, assuming that the new interface provides the correct functionality in a clearer and more concise way?

@crowdmatt
Copy link
Contributor

Willing to break current interface for a cleaner set of code. As long as
we don't regress in terms of functionality. Marshal & Unmarshal in
particular as I rely heavily on those features.

I'm not a huge fan of the way the old /exp/ directory structure was setup
-- honestly, everything is "exp" in a way.

Best,

Matt

Matthew Moore
Co-Founder & iCEO, CrowdMob Inc.
Mobile: (650) 888-5962

Need to schedule a meeting? Invite me via Google Calendar!
matt@crowdmob.com

On Fri, Aug 30, 2013 at 8:29 AM, Ken Walters notifications@github.comwrote:

How willing are you to accept aggressive changes to the dynamodb library
that change the exposed interface? Obviously changing or removing exported
types and functions would impact anyone currently using this package, but
the current state of this library feels pretty incomplete and like it
should probably be in the /exp/ directory at the moment.

Before I make major changes to the library and submit pull requests, how
willing are you to break from the current interface, assuming that the new
interface provides the correct functionality in a clearer and more concise
way?


Reply to this email directly or view it on GitHubhttps://github.com/crowdmob/goamz/issues/16
.

@lostghost
Copy link
Contributor Author

Agreed. The ability to marshal and unmarshal types directly into and out of Dynamo is critical.

Thanks,
Ken

zined pushed a commit to zined/goamz that referenced this issue Mar 23, 2015
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

No branches or pull requests

2 participants