Cijo Thomas

Results 1310 comments of Cijo Thomas

Happy to host it in the contrib repo. (Or even this repo.) Feel free to send a PR.

https://github.com/open-telemetry/opentelemetry-rust/pull/3085 should take care of this.

Can you elaborate the use case more, about fixing descriptions.?

Correct. Persistence storage is not a blocker for stable release. Its a nice-to-have capability, and is not spec-ed yet.

> It's not clear to me how Span.RecordException will work in the SDK if it has to create a log. Will creating a tracer provider require passing a logging provider?...

Yes this is indeed a bug. This was only fixed for ObsGauges, not for Counters/UpDownCounters.

> This might have ramification on how or if this can be pushed to crates.io, I suspect. Should we consult with someone with more knowledge of the cargo publishing process...

> Is there some related issue with the discussion about the rename? The naming consistency should be fixed indeed, but I'd like to read about why you picked to normalize...

Closing now, keeping a new issue to track this: https://github.com/open-telemetry/opentelemetry-rust/issues/2826