-
Notifications
You must be signed in to change notification settings - Fork 132
initTracer should pass options.traceId128bit #397
initTracer should pass options.traceId128bit #397
Conversation
Codecov Report
@@ Coverage Diff @@
## master #397 +/- ##
=========================================
Coverage ? 98.65%
=========================================
Files ? 50
Lines ? 2010
Branches ? 377
=========================================
Hits ? 1983
Misses ? 27
Partials ? 0
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
src/configuration.js
Outdated
@@ -191,6 +191,8 @@ export default class Configuration { | |||
* @param {Object} [options.logger] - a logger (see ./_flow/logger.js) | |||
* @param {Object} [options.tags] - set of key-value pairs which will be set | |||
* as process-level tags on the Tracer itself. | |||
* @param {boolean} [options.traceId128bit] - generate root span with a 128bit traceId. | |||
* BaggageRestrictionManager API from ./baggage.js. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops )
Remove this line
Signed-off-by: Aleksei Androsov <[email protected]>
1ea07ae
to
ad6ce57
Compare
* fix(plugin-http): ensure no leaks closes jaegertracing#397 Signed-off-by: Olivier Albertini <[email protected]> * fix: add @Flarna recommandations Signed-off-by: Olivier Albertini <[email protected]>
Which problem is this PR solving?
128 bit traceid was added at #361
But I can't do like this
const tracer = require('jaeger-client').initTracer(config, { traceId128bit: true });
.Short description of the changes
options.traceId128bit
tonew Tracer