-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[processor/transform] Allow using trace_state with key-value struct. #11029
[processor/transform] Allow using trace_state with key-value struct. #11029
Conversation
9232dba
to
bdbd456
Compare
bdbd456
to
6a5f663
Compare
6a5f663
to
5d713cc
Compare
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.
Thanks for the contribution!
5d713cc
to
c2e84c5
Compare
@TylerHelmuth Thanks for your suggestions, I had resubmitted. |
c2e84c5
to
70ff44a
Compare
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.
small nits
70ff44a
to
158db51
Compare
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.
Thanks!
@ralphgj please take a look at the conflicts |
b5dac3e
to
24f79d8
Compare
@TylerHelmuth I had resolved it, please review again. Thanks! |
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.
Thanks!
Description:
resolve #10849
Link to tracking Issue:
Testing:
Documentation: