-
Notifications
You must be signed in to change notification settings - Fork 327
Set SpanKind for HTTP and gRPC requests #614
Conversation
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.
Cool, just some minor typo.
plugin/ocgrpc/client.go
Outdated
@@ -25,6 +25,8 @@ import ( | |||
// traces. Use with gRPC clients only. | |||
type ClientHandler struct { | |||
// StartOptions allows configuring the StartOptions used to create new spans. | |||
// | |||
// SpanKind will always set to SpanKindClient for spans started with this handler. |
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.
will always be set to SpanKindClient for spans started by
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.
Done
plugin/ocgrpc/server.go
Outdated
@@ -46,6 +46,8 @@ type ServerHandler struct { | |||
// ensures that each service has some opportunity to be traced. If you would | |||
// like to not add any additional traces for this gRPC service, set: | |||
// StartOptions.Sampler = trace.ProbabilitySampler(0.0) | |||
// | |||
// SpanKind will always set to SpanKindServer for spans started from this handler. |
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.
ditto
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.
Done
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, thank you @rakyll!
No description provided.