Skip to content

Commit

Permalink
Update python jaeger export to mention both thrift and proto (#1566)
Browse files Browse the repository at this point in the history
* Update python jaeger export to mention both thrift and proto

* Update content/en/docs/instrumentation/python/exporters.md

Co-authored-by: Srikanth Chekuri <[email protected]>

* Update content/en/docs/instrumentation/python/exporters.md

Co-authored-by: Srikanth Chekuri <[email protected]>

Co-authored-by: Srikanth Chekuri <[email protected]>
  • Loading branch information
cartermp and srikanthccv authored Jul 28, 2022
1 parent 2982f7c commit decd435
Showing 1 changed file with 10 additions and 11 deletions.
21 changes: 10 additions & 11 deletions content/en/docs/instrumentation/python/exporters.md
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,14 @@ Next, install the Jaeger exporter package:
$ pip install opentelemetry-exporter-jaeger
```

Then you can configure the exporter when initializing tracing:
This will install packages for both:

* `opentelemetry-exporter-jaeger-thrift`
* `opentelemetry-exporter-jaeger-proto-grpc`

You can use either to export your traces to Jaeger.

Once the package is installed, you can configure the exporter when initializing tracing:

```python
from opentelemetry import trace
Expand All @@ -182,18 +189,10 @@ trace.set_tracer_provider(provider)
# Merrily go about tracing!
```

### Using Thrift

If you'd prefer to use Thrift as the protocol, you can install the package:

```console
$ pip install opentelemetry-exporter-jaeger-thrift
```

And replace the `JaegerExporter` import declaration with the following:
The previous example uses thrift. To use protobuf, change the import declaration to:

```python
from opentelemetry.exporter.jaeger.thrift import JaegerExporter
from opentelemetry.exporter.jaeger.proto.grpc import JaegerExporter
```

## Zipkin
Expand Down

0 comments on commit decd435

Please sign in to comment.