Skip to content
This repository was archived by the owner on Dec 23, 2023. It is now read-only.

Tags: Add TagContextBuilder.putLocal().#1851

Merged
songy23 merged 1 commit intocensus-instrumentation:masterfrom
songy23:put-local-tag
Apr 19, 2019
Merged

Tags: Add TagContextBuilder.putLocal().#1851
songy23 merged 1 commit intocensus-instrumentation:masterfrom
songy23:put-local-tag

Conversation

@songy23
Copy link
Copy Markdown
Contributor

@songy23 songy23 commented Apr 19, 2019

Fixes #1849.

Add a convenient API so that users don't need to put a tag metadata over and over again when putting tags.

Copy link
Copy Markdown
Contributor

@rghetia rghetia left a comment

Choose a reason for hiding this comment

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

LGTM.
wouldn't it make sense to add putPropagating also?

@songy23
Copy link
Copy Markdown
Contributor Author

songy23 commented Apr 19, 2019

wouldn't it make sense to add putPropagating also?

Make sense to me, will add in a separate PR.

@songy23 songy23 merged commit 7dc2112 into census-instrumentation:master Apr 19, 2019
@songy23 songy23 deleted the put-local-tag branch April 19, 2019 15:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Undeprecate TagContextBuilder and Tag's put without TagMetadata

4 participants