Skip to content

Commit

Permalink
Fix GettingStarted documentation for Jaeger HTTP exporter (#1347) (#1439
Browse files Browse the repository at this point in the history
)
  • Loading branch information
marcalff committed Jun 10, 2022
1 parent 4c657a1 commit 6ee70a7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/public/sdk/GettingStarted.rst
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,9 @@ OpenTelemetry offers six tracing exporters out of the box:
opentelemetry::exporter::jaeger::JaegerExporterOptions opts;
opts.transport_format = opentelemetry::exporter::jaeger::TransportFormat::kThriftHttp;
opts.endpoint = "localhost";
opts.server_port = 6831;
opts.server_port = 14268;
opts.headers = {{}}; // optional headers
auto jaeger_udp_exporter =
auto jaeger_http_exporter =
std::unique_ptr<sdktrace::SpanExporter>(new opentelemetry::exporter::jaeger::JaegerExporter(opts));
Expand Down

1 comment on commit 6ee70a7

@github-actions
Copy link

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'OpenTelemetry-cpp sdk Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 6ee70a7 Previous: 4c657a1 Ratio
BM_BaselineBuffer/1 1397972.5600997808 ns/iter 562812.089920044 ns/iter 2.48
BM_LockFreeBuffer/1 3405216.693878174 ns/iter 610525.1312255859 ns/iter 5.58

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.