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
When we built the context manager we had a legacy async local. Since legacy is removed we should update the context manager to be able to store more than just the active segment. Right now we have no easy way to arbitrarily store context info with context manager without stomping on the segment. this is because in transaction/tracer/index.js we assume all calls to contextManger.getContext() are only the active segment. Once we update we should update the work done in #2437 to use this instead of its own instance of AsyncLocalStorage. We could potentially use this to store other instrumentation data that we now do via symbols and a lot of roundabout instrumentation.
Acceptance Criteria
Update context manager to allow storage of more than just the active segment and update places that call contextManager.getContext() to properly extract the active segment in those instances
The text was updated successfully, but these errors were encountered:
I was looking into this for a different project. This branch updates context manager to add a new getSegment method. I still need to update to allow to mix-in other contexts
Description
When we built the context manager we had a legacy async local. Since legacy is removed we should update the context manager to be able to store more than just the active segment. Right now we have no easy way to arbitrarily store context info with context manager without stomping on the segment. this is because in
transaction/tracer/index.js
we assume all calls tocontextManger.getContext()
are only the active segment. Once we update we should update the work done in #2437 to use this instead of its own instance of AsyncLocalStorage. We could potentially use this to store other instrumentation data that we now do via symbols and a lot of roundabout instrumentation.Acceptance Criteria
Update context manager to allow storage of more than just the active segment and update places that call
contextManager.getContext()
to properly extract the active segment in those instancesThe text was updated successfully, but these errors were encountered: