# Passing context to another thread

**URL:** <https://discuss.elastic.co/t/passing-context-to-another-thread/349055>\
**Category:** APM\
**Tags:** java\
**Created:** [December 11, 2023, 2:39pm UTC](https://discuss.elastic.co/t/passing-context-to-another-thread/349055 "2023-12-11T14:39:57Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![johngregg](https://avatars.discourse-cdn.com/v4/letter/j/e19b73/32.png) [@johngregg](https://discuss.elastic.co/u/johngregg)\
**Post date:** [December 11, 2023, 2:39pm UTC](https://discuss.elastic.co/t/passing-context-to-another-thread/349055/1 "2023-12-11T14:39:57Z")

</div>

I am using the 1.42.0 java agent.

I am working on an app that uses an unsupported threading library. Based on reading some posts here, we decided to use the OTel library like this:

```auto
import io.opentelemetry.api.trace.Span;

public class MyRunnable implement Runnable {
  private Span span;

  public MyRunnable() {
    this.span = Span.current();
  }

  public void run() {
    try (Scope scope = span.makeCurrent()) {
      // do some backend call here
    }
  }
}

```

I think the actual Span implementation that gets returned from Span.currentSpan() is co.elastic.apm.agent.opentelemetry.tracing.OTelSpan.

This seems to work ok in the sense that the backend calls are now associated with the trace that was started on a parent thread.

However the stack traces are mixed up. For example, often I will see a span for an Oracle stored proc with stack trace from Apache HttpClient.

Any idea what I'm doing wrong?

thanks

---

<div class="post-metadata">

**Author:** ![Sylvain\_Juge](https://sea2.discourse-cdn.com/elastic/user_avatar/discuss.elastic.co/sylvain_juge/32/55521_2.png) [@Sylvain\_Juge](https://discuss.elastic.co/u/Sylvain_Juge)\
**Post date:** [December 11, 2023, 2:56pm UTC](https://discuss.elastic.co/t/passing-context-to-another-thread/349055/2 "2023-12-11T14:56:34Z")

</div>

Hi,

Here you capture the current span in the constructor, but you should instead capture and store the `Context.current()` instead.

Then, when you execute the `run()` method, you should make the stored context the current one.

The difference here is that due to asynchronous execution the span for which you store a reference might be already ended when you make it current, whereas a context is just a context and just provides the context propagation.

What also needs to be checked here is that all the calls to the constructor are within the expected scope (when there is already a parent span), as the calls to the constructor become caller-sensitive.

---

<div class="post-metadata">

**Author:** ![johngregg](https://avatars.discourse-cdn.com/v4/letter/j/e19b73/32.png) [@johngregg](https://discuss.elastic.co/u/johngregg)\
**Post date:** [December 11, 2023, 3:12pm UTC](https://discuss.elastic.co/t/passing-context-to-another-thread/349055/3 "2023-12-11T15:12:19Z")

</div>

Thanks Sylvain. Can you give an example of the last part? I don't know what you mean about checking the expected scope.

---

<div class="post-metadata">

**Author:** ![Sylvain\_Juge](https://sea2.discourse-cdn.com/elastic/user_avatar/discuss.elastic.co/sylvain_juge/32/55521_2.png) [@Sylvain\_Juge](https://discuss.elastic.co/u/Sylvain_Juge)\
**Post date:** [December 11, 2023, 3:52pm UTC](https://discuss.elastic.co/t/passing-context-to-another-thread/349055/4 "2023-12-11T15:52:09Z")

</div>

What I meant by that is that when the constructor is called `Context.current()` returns something different than the value of `Context.root()`, otherwise there is nothing to propagate, and the spans that will be created from that will be top-level spans.

In your application this might be implicit, if you get top-level spans where you expect to have child spans of a parent span, then that could be the issue here.

In short, try it by just propagating the context, I'm probably overthinking what could be wrong here :-).

---

<div class="post-metadata">

**Author:** ![system](https://us1.discourse-cdn.com/elastic/original/3X/1/a/1ac57faf039f6b580b3f104ef42a2a89e41014de.png) [@system](https://discuss.elastic.co/u/system)\
**Post date:** [January 8, 2024, 3:52pm UTC](https://discuss.elastic.co/t/passing-context-to-another-thread/349055/5 "2024-01-08T15:52:35Z")

</div>

This topic was automatically closed 28 days after the last reply. New replies are no longer allowed.