stillalex commented on code in PR #1898:
URL: https://github.com/apache/solr/pull/1898#discussion_r1319873973


##########
solr/solr-ref-guide/modules/deployment-guide/pages/distributed-tracing.adoc:
##########
@@ -25,6 +25,17 @@ A sampled distributed tracing query request displayed in 
Jaeger UI looks like th
 .Tracing of a Solr query
 image::distributed-tracing/query-request-tracing.png[]
 
+== Always-on Trace Id generation
+
+Starting from 9.4 Solr ships with an always-on trace id generator. This will 
inject trace id headers for every Solr request, propagating the client supplied 
value or generating a new id as needed and replaces the existing `rid` 
mechanism.
+
+The header name it uses for propagation is `X-Trace-Id`, this can be changed 
by updating the system property `solr.traceIdHeader`.
+
+This plugin will not be enabled if another Tracing mechanism is setup via Solr 
config.
+This plugin can be disabled by setting the `solr.alwaysOnTraceId` system 
property to `false`.
+
+Note: Solr 9.4 ships with an `OpenTracing` based lightweight `Tracer` 
implementation and Solr 10 has an updated `OpenTelemetry` based 
`TextMapPropagator` implementation.
+

Review Comment:
   paragraph updated. please let me know if it looks better



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to