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
Is your feature request related to a problem? Please describe.
In #1620 we are looking at implementing parallel processing for batches to reduce clock time spent handling batch requests. As part of this, we have started considering what happens when we rely on thread-local storage to propagate request context. In the batch case this affects immediately tracing, but more broadly, I think it is worth considering how PT might help support multi-threaded processing, or at the very least, avoid complicating it.
I would suggest the next step here is to consider all areas of PT that use thread local, and if that can be avoided. This may very well be v2 "cleanup style" work. Next, what other areas of PT may benefit from supporting multi-threaded processing, and what would the impact of this be?
Describe the solution you'd like
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
In #1620 we are looking at implementing parallel processing for batches to reduce clock time spent handling batch requests. As part of this, we have started considering what happens when we rely on thread-local storage to propagate request context. In the batch case this affects immediately tracing, but more broadly, I think it is worth considering how PT might help support multi-threaded processing, or at the very least, avoid complicating it.
I would suggest the next step here is to consider all areas of PT that use thread local, and if that can be avoided. This may very well be v2 "cleanup style" work. Next, what other areas of PT may benefit from supporting multi-threaded processing, and what would the impact of this be?
Describe the solution you'd like
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: