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

Fix TraceDoer so it fetches the trace details properly. #1129

Merged
merged 1 commit into from Mar 17, 2017

Conversation

Projects
None yet
1 participant
@raphael
Copy link
Member

raphael commented Mar 17, 2017

Do it at call time so that the same traced doer can be reused to do many
requests potentially concurrently.

Fix TraceDoer so it fetches the trace details properly.
Do it at call time so that the same traced doer can be reused to do many
requests potentially concurrently.

@raphael raphael merged commit 7bd2b30 into v1 Mar 17, 2017

3 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@raphael raphael deleted the v1.concurrent_trace branch Mar 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment