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.

Sign up for GitHub

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

Jump to bottom

C++ opencensus filter: Fix point of creating context for overall call #27221

Merged
merged 1 commit into from
Sep 2, 2021
Merged

C++ opencensus filter: Fix point of creating context for overall call #27221

merged 1 commit into from
Sep 2, 2021

Conversation

yashykt
Copy link
Member

@yashykt yashykt commented Sep 1, 2021 •
edited

Fix #27013

@yashykt yashykt added release notes: yes Indicates if PR needs to be in release notes lang/c++ labels Sep 1, 2021
markdroth
markdroth approved these changes Sep 1, 2021
View reviewed changes
Copy link
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this!

@yashykt yashykt merged commit 9e4c657 into grpc:master Sep 2, 2021
markdroth pushed a commit to markdroth/grpc that referenced this pull request Sep 2, 2021
@yashykt @markdroth
C++ opencensus filter: Fix point of creating context for overall call (…
d69a17a 
…grpc#27221)
@markdroth markdroth mentioned this pull request Sep 2, 2021
Closed
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Sep 2, 2021
yashykt added a commit to yashykt/grpc that referenced this pull request Sep 2, 2021
@yashykt
C++ opencensus filter: Fix point of creating context for overall call (…
23ba535 
…grpc#27221)
@yashykt yashykt mentioned this pull request Sep 2, 2021
Merged
srini100 pushed a commit that referenced this pull request Sep 3, 2021
@yashykt
C++ opencensus filter: Fix point of creating context for overall call (…
ab6beb3 
…#27221) (#27238)
lidizheng pushed a commit to lidizheng/grpc that referenced this pull request Sep 23, 2021
@yashykt @lidizheng
C++ opencensus filter: Fix point of creating context for overall call (…
b4dbfd5 
…grpc#27221)
@yashykt yashykt mentioned this pull request Sep 29, 2021
Merged
@yashykt yashykt deleted the Fix27013 branch May 18, 2023 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markdroth markdroth markdroth approved these changes

Assignees
No one assigned
Labels
imported Specifies if the PR has been imported to the internal repository lang/c++ priority/P0/RELEASE BLOCKER release notes: yes Indicates if PR needs to be in release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Census context doesn't behave as expected.
2 participants
@yashykt @markdroth