[Bug]: TracerProvider::force_flush moved from API to SDK without any way to downcast GlobalTracerProvider about opentelemetry-rust HOT 9 CLOSED

Cloning the provider before passing it to set_tracer_provider works. Thanks again!

from opentelemetry-rust.

https://github.com/open-telemetry/opentelemetry-rust/blob/main/examples/tracing-jaeger/src/main.rs#L46 Could you see if this is what you are looking for?

from opentelemetry-rust.

@cijothomas: Thanks for the quick reply!

The use case here is FaaS/"serverless" (e.g., AWS Lambda). Specifically, we need a way to flush traces after a request completes, but before AWS Lambda freezes the container to wait for the next request (which may never come, in which case unflushed traces are lost). Prior to the 0.18 release, I would have called opentelemetry::global::tracer_provider().force_flush() here, but that's no longer possible.

Shutting down and registering a new tracing provider for each request seems like it would introduce a lot of unnecessary overhead, when we just need access to the force_flush method.

from opentelemetry-rust.

Also, I'm willing to contribute a PR to get this done once there's consensus on the API changes.

from opentelemetry-rust.

@ramosbugs how/where is the tracer provider being created in this case?

from opentelemetry-rust.

I'm using the following code when the Lambda process starts (i.e., from main), before it enters its request loop:

let batch = BatchSpanProcessor::builder(exporter, opentelemetry::runtime::Tokio)
  .with_max_queue_size(MAX_SPAN_QUEUE_SIZE)
  .build();

let provider = TracerProvider::builder()
  .with_span_processor(batch)
  .with_config(tracer_config)
  .build();

opentelemetry::global::set_tracer_provider(provider);

from opentelemetry-rust.

One possible solution is to impl<T> TracerProvider for Arc<T> where T: TracerProvider. Then, we can just pass a copy of the Arc<opentelemetry_sdk::trace::TracerProvider> to set_tracer_provider and maintain our own copy for access to force_flush.

I don't see anything that requires a mutable reference to the provider, so an Arc seems like it should work?

from opentelemetry-rust.

TracerProvider is already clone, so you don't need the Arc, you can pass a clone to set_tracer_provider directly 👍

from opentelemetry-rust.

Oh, I totally missed that! Thank you, I will confirm that this works and then close the issue or report back otherwise.

from opentelemetry-rust.