-
Notifications
You must be signed in to change notification settings - Fork 44
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
Log the original trace for the actor #326
Conversation
8ecc167
to
be66e4e
Compare
be66e4e
to
992d43a
Compare
This helps a lot figuring out where the particular errors are coming from.
992d43a
to
403a2ba
Compare
The ruby 2.2 is failing due to missing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left one comment inline, I'm ok with dropping support for 2.2
Can we get this in? |
I thought we're dropping support for 2.2 before this gets in |
Required by Dynflow#326
You're right. Dropped. Feels good to remove all the conditions checking for older ruby version :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
APT
Thanks @iNecas ! |
This helps a lot figuring out where the particular errors are coming
from.
Before - long backtrace that doesn't lead to the original caller:
After - the concurrent-ruby part is filtered out, and the original backtrace back to the initiator
is included, including jumps though multiple actors:
FYI there is a track to propagate the backtraces within concurrent-ruby executor here ruby-concurrency/concurrent-ruby#810