-
Notifications
You must be signed in to change notification settings - Fork 195
Upgrade to SDK 3.11 #2116
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
Upgrade to SDK 3.11 #2116
Conversation
…iers.ATTR_OPERATION Signed-off-by: Emilien Bevierre <emilien.bevierre@couchbase.com>
84caa5b to
afe8d4d
Compare
| CoreTransactionAttemptContext internal = ctxOpt.get().getCore(); | ||
| RequestSpan span = CbTracing.newSpan(internal.core().context(), TRANSACTION_OP_INSERT, | ||
| internal.span()); | ||
| span.attribute(TracingIdentifiers.ATTR_OPERATION, TRANSACTION_OP_INSERT); |
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.
Just curious, I see the ATTR_OPERATION span attributes have been removed but I don't see any reasoning in the release notes.
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.
onobc
left a comment
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.
LGTM. Thanks for the update!
| CoreTransactionAttemptContext internal = ctxOpt.get().getCore(); | ||
| RequestSpan span = CbTracing.newSpan(internal.core().context(), TRANSACTION_OP_INSERT, | ||
| internal.span()); | ||
| span.attribute(TracingIdentifiers.ATTR_OPERATION, TRANSACTION_OP_INSERT); |
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.
onobc
left a comment
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.
Oh, one post-review comment @emilienbev...
Do you mind adding yourself as @ author on the modified files?
Signed-off-by: Emilien Bevierre <emilien.bevierre@couchbase.com>
dcc5fd7 to
dde0290
Compare
|
Thanks @onobc ! |
Remove deprecated and redundant
TracingIdentifiers.ATTR_OPERATION.