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.

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

OpenCensus: Move measures, views and CensusContext to include file #31341

Merged
merged 8 commits into from Oct 21, 2022

Conversation

yashykt
Copy link
Member

@yashykt yashykt commented Oct 13, 2022

Proposal in grpc/proposal#334

Note that these are moved under the grpc::experimental namespace in the include file, but the original header files in src maintain using declarations in the grpc namespace for all of them so that, so that existing users of these APIs don't get broken.

src/cpp/ext/filters/census/grpc_plugin.cc Outdated Show resolved Hide resolved
@yashykt yashykt merged commit 3b6675c into grpc:master Oct 21, 2022
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Oct 21, 2022
@yashykt yashykt deleted the OpenCensusPublicHeaders branch May 18, 2023 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bloat/none imported Specifies if the PR has been imported to the internal repository lang/c++ per-call-memory/neutral per-channel-memory/neutral release notes: yes Indicates if PR needs to be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants