-
Notifications
You must be signed in to change notification settings - Fork 825
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
Remove dependency to all exporters in SDK #4150
Comments
Agreed. I think we should also first provide a heads-up notice one release prior, as was done in #4068 |
How would the environment variables then work? Do we make these exporters optional peer dependencies? |
with the Jaeger exporter, we currently lazy-require it. we could do something similar with the otlp exporters. IIRC npm 7+ installs optional peer dependencies by default when installing the package, so that might not be an option. |
if someone sets an env var to use I assume config via env should be anyway that generic that it works with any exporter, not just a limited list of predefined ones. |
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
I'm removing |
Followup on #4145
I think we should remove the dependency to all exporters from SDK similar as it was done for Jaeger in #4049.
The OTLP exporters include quite some transitive dependencies like grpc which are often not needed. This results in unneeded long installation time, unneeded high disc size/bundle size.
There might be more like longer startup time, higher memory consumption just because unneeded stuff is read and code is held in memory.
The text was updated successfully, but these errors were encountered: