WenTao Ou

Results 226 comments of WenTao Ou

Good point, I can take this several days later when I have time. We can deny unknown types or some well-known types in type traits.

> Thanks for the fix. > > See suggested change. > > Not sure why opentelemetry-proto.cmake was not formatted in my previous change, as CI was ok on format. Maybe...

What's your version of otel-cpp and do you enable async exporting? There was a thread safety problem before 1.10.0 in OTLP HTTP exporter when otel-cpp is built without async export(Without...

> @owent - 1.15, haven't enabled async exporting... is async export still in preview in 1.15? gRPC async exporting is still in preview.

Does this problem happens when shuting down? Do you compile both otel-cpp and proto as dynamic library?Just wondering why the destructor of `RepeatedPtrField` is in gRPC exporter.

> It's HTTP exporter, and proto is from yum install. > > We're investigating if it's memory corruption from somewhere else. Do you mean protobuf? I reviewed the codes and...

I found another crash in #2982 when using metrics and timeout happens. Not sure if it relates this one.

@michalpristas Could you try main branch or #2983 ? Some `std::async` implementations of STLs may have bugs and crash sometimes, this PR replace these APIs with the more stable one....

> Thanks for the report. > > Using opentelemetry-1.16.0 with `WITH_ASYNC_EXPORT_PREVIEW=OFF`: can not reproduce > > Using opentelemetry-1.16.0 with `WITH_ASYNC_EXPORT_PREVIEW=ON`: reproduced > > The example code `example_otlp_http` invokes `ForceFlush()`, which...

Could you update otel-cpp to the latest version? The latest API component no longer requires Abseil, even with OTLP exporters enabled. The old versions require `-DWITH_ABSEIL=ON` when you enable any...