-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Spanner: getting the unsupported sampler while keeping the session alive #7100
Comments
Thanks @harshit-bci for letting us know about this. Error particularly is coming from the bridge code https://github.com/open-telemetry/opentelemetry-go/blob/1f5e6adbf2ca8fdb394ea07b3370506889a10bfa/bridge/opencensus/internal/oc2otel/tracer_start_options.go#L43, Let me try to find the fix to support OpenTelemetry in Spanner. We already have a open ticket to support OpenTelemetry for traces here #4237. |
Back to your question on handling sessions, Go client library is handling the sessions fine in your application, its just not able to trace a specific method using OpenTelemetry, I think the error reported in issue is safe to ignore and should not be impacting any database operations. |
@rahul2393 By when this can be resolved ? The ticket #4237 (that you mentioned) is already open from last 5 months. |
@harshit-bci The issue is happening because of OpenTelemetry limitation here https://github.com/open-telemetry/opentelemetry-go/tree/main/bridge/opencensus#incompatibilities to support samplers. While we are looking into possible solutions out there but it would make more sense if you open a ticket here https://github.com/open-telemetry/opentelemetry-go/issues/new?assignees=&labels=bug&template=bug_report.md&title= because we don't support OpenTelemetry in Go Client libraries yet. |
This fix seems like dupe of #4237 so I will close out this issue. The implementations for most OpenTelemetry services are not yet stable. This is especially true in Go where tracing is currently stable, metrics are alpha, and logging does not exist. As the project matures over time we will continue to evaluate if it is right to integrate with it. |
I met this issue too. |
Client
go-spanner
Environment
GKE
Go Environment
$ go version
1.18.1
Using https://pkg.go.dev/cloud.google.com/go/spanner v1.30.0
Code
go library for Spanner suggests that there is no need to handle session on our own and we are not doing any of the thing on our own. It's the
HealthCheckWorker
which is doing all of the things on it's own.Expected behavior
There should not be any error while executing
Select 1
.Actual behavior
Getting Below error
starting span "cloud.google.com/go/spanner.ping": unsupported sampler: 0x9f1c00
Below is the traceback
Additional context
This issue seems to be fixed here but still we are getting this issue.
The text was updated successfully, but these errors were encountered: