-
Notifications
You must be signed in to change notification settings - Fork 170
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
Fixes can't add a new key into hash during iteration
error
#174
Conversation
Thanks @ankane - Think you could add the test case to the cache_spec? |
Hey @bleonard, thanks for the response. Added a test case. |
@ankane - I've added you as a member if you'd like to participate with makara. Seems like a good idea if you are depending on it. If so, want to test out master and tag a new release, etc? |
Yeah, happy to help out. I'll defer to you on when to release new versions. We'll try to get on master in the near future. |
Hey @bleonard, looks like I was invited to join org rather than this specific project. |
It was a team that was only on makara projects. Now given project access if that's better for you. |
Ah, gotcha. All good now. Thanks. |
Interesting that Ruby 2 is freaking out (no log) on Travis. https://travis-ci.org/taskrabbit/makara/builds/280541383 Some sort of Mutex or thread difference? |
That's odd. Can you try restarting the jobs? |
Actually the nextval one (#173) failed first all in Ruby 2. Then this one, so likely not this one. Likely not that one either based on it's simplicity. |
https://travis-ci.org/taskrabbit/makara/jobs/280541385 bundler failing to install in that ruby. tried similar setup locally and it worked. |
This adds thread-safety to the
MemoryStore
. Here's code to reproduce the current issue.