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

Added SpanKind for gRPC Server and Client #25

Merged
2 commits merged into from
Oct 21, 2019

Conversation

siniy123
Copy link
Contributor

No description provided.

contrib/google.golang.org/grpc/client.go Outdated Show resolved Hide resolved
contrib/google.golang.org/grpc/client.go Show resolved Hide resolved
contrib/google.golang.org/grpc/server.go Outdated Show resolved Hide resolved
@mpetazzoni mpetazzoni assigned rmfitzpatrick, ghost and keitwb and unassigned rmfitzpatrick and ghost Oct 18, 2019
@siniy123
Copy link
Contributor Author

I removed SpanKind from Stream methods. SpanKind is only set in the interceptors.

Copy link
Contributor

@rmfitzpatrick rmfitzpatrick left a comment

Choose a reason for hiding this comment

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

This seems good to me.

@ghost
Copy link

ghost commented Oct 21, 2019

We haven't done a full pass to make the module supported but this change lgtm.

@ghost ghost closed this Oct 21, 2019
@ghost ghost reopened this Oct 21, 2019
@ghost ghost merged commit 29320dc into signalfx:master Oct 21, 2019
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants