You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
marcalff
added
triage/accepted
Indicates an issue or PR is ready to be actively worked on.
and removed
needs-triage
Indicates an issue or PR lacks a `triage/foo` label and requires one.
labels
Jan 27, 2025
So far deployed it internally to dozen of our game tools in order to track the status of an homebrew http caching server. Hasn't had any crashes directly related to it, and no one has experience any difference in performance, but possibly there might be a bit better ways to optimize these further:
- Performance optimizations - Less threads being used, less copying, allocations & deallocations
I've been using my own soft fork, which I merge often to the main one, and I've found that producing single .dll (for all three major platforms), and additional symbol files gives me easiest way to deploy it. I wonder where the header-only library approach is going, there are various singletons and these are tricky to have without actual binary body, it was not listed directly, but maybe as part of packaging. Packing api+sdk in single .dll fixes it (but could've worked in two, or more), but as I said, I didn't want to add extra "noise" to the rest of the team. Especially since we place all our binaries in single "bin" folder, and having (just making this up) - something like zlib.dll or curl.dll that may come from compiling from vcpkg may interfere with same named .dll in the folder. Static linking is also problematic as it creates opportunities for ODR violations, and I can't use delay loading with which I completely disable even loading the library. Hence:
- Packaging - precompiled dynamic libraries for Windows, Linux, OSX might be good to have exposing only `opentelemetry::` symbols.
Happy 2025
This is a user survey, please indicate in comments what you think opentelemetry-cpp should focus on in priority this year.
Possible areas include:
Feel free to also suggest items not in this list.
The text was updated successfully, but these errors were encountered: