-
-
Notifications
You must be signed in to change notification settings - Fork 608
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
Trampoline chained DBIOActions #1703
Conversation
@szeiger I recommend reading the diff ignoring whitespace like this: https://github.com/slick/slick/pull/1703/files?w=1 |
I made an update to this PR to only trampoline when recursion gets high. That should remove any worries about performance. |
e1e33bd
to
7ae5f53
Compare
@szeiger Did you have time to check this? |
@derekjw can you rebase this on current master |
@hvesalai sure can, I should be able to get that done this evening |
7ae5f53
to
70dfd2f
Compare
@hvesalai Rebased onto current master, travis build failed though (I ran tests multiple times locally and can't get a failure). Looks like travis is giving it another try now |
Yes, I'm seeing that problem sporadically. Not related to your PR |
LGTM |
@hvesalai thanks! |
Fix for #1700
Previous fix #1701 did not work in the case of other actions being interwoven with the flatMap actions. A more general solution uses a trampoline for calls to runInContext, using DBIO.sameThreadExecutionContext.