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

Implement and test BatchPutItem #165

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattbornski
Copy link

I have a use case which would be more optimal with batch inserts, so I have implemented it in your driver. Enjoy.

@m5rk
Copy link

m5rk commented Dec 12, 2013

@mattbornski How do you envision this new write capability being accessed from a dynamoid model? We would also like to utilize the batch insert.

@mattbornski
Copy link
Author

I imagine that we'll have to implement something like this:

https://github.com/zdennis/activerecord-import

but for dynamoid. Syntax along the lines of
"YourModel::import([YourModel.new(foo, bar), YourModel.new(foo2, bar2)])".

On Thu, Dec 12, 2013 at 10:44 AM, Mark McEahern [email protected]:

@mattbornski https://github.com/mattbornski How do you envision this
new write capability being accessed from a dynamoid model? We would also
like to utilize the batch insert.


Reply to this email directly or view it on GitHubhttps://github.com//pull/165#issuecomment-30449489
.

@lisad
Copy link
Contributor

lisad commented Dec 12, 2013

This should also be hooked into ActiveRecord's "subset conditions" (http://guides.rubyonrails.org/active_record_querying.html, section 2.3.3) which would be more handily named "batch finds"

I'm guessing you can use that to say "Client.where(:id => [1, 3, 5]).update_all({ :paid_up => true } "

@mattbornski
Copy link
Author

@lisad feel like taking a stab at that? The batch insertions was really the only batch operation I needed.

pkaeding pushed a commit to pkaeding/Dynamoid that referenced this pull request May 31, 2014
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.

3 participants