Cijo Thomas

Results 1310 comments of Cijo Thomas

> @cijothomas I see you mentioned that exporter hangs with tokio::main. Was it because of the infinite loop scenario similar to https://github.com/open-telemetry/opentelemetry-rust/issues/1745 Yes exactly. Its deadlock caused by reentrancy in...

> > You are right! Added #2199 recently to prove this with tests, so the fix can be easily validated. Filtering is a temp/hack solution - this has to be...

I think this can be closed with https://github.com/open-telemetry/opentelemetry-rust/pull/2394 which shows how to use OTLP/gRPC with SimpleProcessor.

> I'm worried about support for situations where threads may be problematic, either because their are not supported or because or due to the use of functions unsafe to call...

@sandersaares Could you take a look?

This is now merged via alternate PR, Closing. Thanks everyone for thorough reviews!

> Just to be clear - it doesn't work with reqwest client (both blocking and async) ? "reqwest-blocking-client" works fine in this case. (only as long as it is not...

Planning to close the issue with the following: 1. To use simple exporter with otlp/http, reqwest-blocking feature should be used. (which would be the default soon). 2. AND the logs/span...

> > Don't think we need to modify SimpleExporter to spin up its own thread and make this work more easily.... > > Any comments/thoughts on this? > > Moving...