Skip to content
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

driver-sync: instrument with OpenCensus #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

odeke-em
Copy link
Member

@odeke-em odeke-em commented Apr 9, 2018

Work in progress to instrument this driver with OpenCensus.
All tests pass, repro coming up soon too.

driver-sync/com.mongodb.client.internal: ScopedSpans

Use ScopedSpans to enable proper context propagation
instead of starting and ending spans manually.

Thanks to @bogdandrutu for the advice.

@odeke-em odeke-em force-pushed the instrument-with-opencensus branch 3 times, most recently from da8a03e to c439709 Compare April 13, 2018 01:38
@odeke-em odeke-em force-pushed the instrument-with-opencensus branch 4 times, most recently from a8ea645 to 7d4ff9c Compare April 25, 2018 03:30
@odeke-em odeke-em force-pushed the instrument-with-opencensus branch 5 times, most recently from 6f802f8 to 5ccb005 Compare June 26, 2018 00:27
driver-{core, sync}/com.mongodb.client.internal: ScopedSpans

Use ScopedSpans to enable proper context propagation
instead of starting and ending spans manually.

Use `com.mongodb.client.internal.<ClassName>` as the
prefix for the trace.

all: more scopedSpan instrumentation with OpenCensus

driver-sync/*/gridfs: Instrument GridFSBucketImpl

Statically initialized tracer per class that uses it
@odeke-em odeke-em force-pushed the instrument-with-opencensus branch from 5ccb005 to fc2946c Compare June 26, 2018 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant