Skip to content

Commit

Permalink
Minor change: ParentSampler -> ParentBased in Zipkin example descript…
Browse files Browse the repository at this point in the history
…ion (#1171)

* ParentSampler -> ParentBased in Zipkin example description

* update CHANGELOG
  • Loading branch information
matej-g authored Sep 13, 2020
1 parent 6f048eb commit 77de199
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,10 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
recommend the use of `newConfig()` instead of `configure()`. (#1163)
- The `otlp.Config` type has been unexported and changed to `otlp.config`, along with its initializer. (#1163)

### Fixed

- Zipkin example no longer mentions `ParentSampler`, corrected to `ParentBased`. (#1171)

## [0.11.0] - 2020-08-24

### Added
Expand Down
2 changes: 1 addition & 1 deletion example/zipkin/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ func initTracer(url string) {
// Create Zipkin Exporter and install it as a global tracer.
//
// For demoing purposes, always sample. In a production application, you should
// configure the sampler to a trace.ParentSampler(trace.TraceIDRatioBased) set at the desired
// configure the sampler to a trace.ParentBased(trace.TraceIDRatioBased) set at the desired
// ratio.
err := zipkin.InstallNewPipeline(
url,
Expand Down

0 comments on commit 77de199

Please sign in to comment.