[SOLR-16950] SimpleTracer propagation for manual transaction ids

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 9.4
• Component/s: tracing
• Assignee: Alex Deparvu
• Reporter: Alex Deparvu
• Created: 24/Aug/23 19:06
• Updated: 15/Oct/23 20:13
• Resolved: 15/Sep/23 04:01

Description

This only applies to the always-on simple tracer. Being able to pass in a custom trace header and have it correctly propagated across the cluster in all types of requests is a big feature of this component.

I noticed that there are UpdateRequests that will not correctly propagate a manual trace headers, due to the fact that the Solr client can recreate the UpdateRequest without copying the headers over top the new request.
There is a failing test I commented out to showcase the issue https://github.com/apache/solr/blob/ce6c13e41806233daaf25f1a7545c7aad34c169d/solr/core/src/test/org/apache/solr/util/tracing/TestSimplePropagatorDistributedTracing.java#L75

Issue Links

• links to: GitHub Pull Request #1906