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.

Sign up for GitHub

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

Jump to bottom

OpenTelemetry Support #146

Open

dave-tucker opened this issue Jun 7, 2021 · 1 comment · May be fixed by #189

Assignees

Labels

feature

Milestone

Collaborator

dave-tucker commented Jun 7, 2021 •

edited

Loading

I'm really keen on adding support for opentelemetry.io
This would require:

Adding ctx context.Context to all clientAPI calls, which seems sensible for multiple reasons (i.e timeouts)
Creating spans at useful points in the code, so we can see, for example, how long a given transaction takes
We may also want to expose some metrics for the cache

The text was updated successfully, but these errors were encountered:

Contributor

vtolstov commented Jun 7, 2021

nice

Yes, useful also for some logging for example or pass additional info via context and extract it via span tags via some func observer
I think that for each method we may create one span, and fill it via adding event to span so span - one operation that holds info for all processes like decode/encode/sent req/recv rsp.
Yes, will be cool to be able to tune cache based on some metrics..

dave-tucker added the feature label

dave-tucker self-assigned this

dave-tucker linked a pull request

that will close this issue

Add opentelemetry support #189

Open

dave-tucker added this to the 0.7.0 milestone

dave-tucker mentioned this issue

Add configurable logger, metrics #223

Merged

halfcrazy mentioned this issue

Feature Request: Logger per transact #339

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment