Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADMIN] 2025 WISH LIST #3256

Open
7 tasks
marcalff opened this issue Jan 27, 2025 · 1 comment
Open
7 tasks

[ADMIN] 2025 WISH LIST #3256

marcalff opened this issue Jan 27, 2025 · 1 comment
Assignees
Labels
enhancement New feature or request triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@marcalff
Copy link
Member

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:

  • Better coverage of the opentelemetry specification
  • Performance optimizations
  • Internal documentation in the code, like doxygen
  • External documentation, in opentelemetry.io
  • Examples of usage
  • Packaging
  • Platform / Compiler coverage

Feel free to also suggest items not in this list.

@marcalff marcalff added the enhancement New feature or request label Jan 27, 2025
@marcalff marcalff self-assigned this Jan 27, 2025
@marcalff marcalff pinned this issue Jan 27, 2025
@github-actions github-actions bot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Jan 27, 2025
@marcalff 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
@malkia
Copy link

malkia commented Jan 28, 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.

@marcalff marcalff unpinned this issue Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

2 participants