opentelemetry-js
opentelemetry-js copied to clipboard
Remove unused `AsyncHooksContextManager`
Also consider remove `AsyncHooksContextManager` because it will no longer be used.
Originally posted by @dyladan in https://github.com/open-telemetry/opentelemetry-js/issues/3720#issuecomment-1823194259
No longer used in SDK 2.0 so it can be safely removed.
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.
@dyladan I'm currently refining issues, one question:
- should we rename the package
@opentelemetry/context-async-hooks
to@opentelemetry/context-async-local-storage
as it'll be the only context manager that's left in that package?
If yes, then I think we can break this down into multiple subtasks:
- [ ] change all usages of
AsyncHooksContextManager
toAsyncLocalStorageContextManager
- [ ] PENDING: rename package to
@opentelemetry/context-async-local-storage
- [ ] integrate abstract base
AbstractAsyncHooksContextManager
intoAsyncLocalStorageContextManager