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.

Sign up for GitHub

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

Jump to bottom

Thread Context should collect unique response headers in async actions #19222

Closed
pickypg opened this issue Jul 1, 2016 · 2 comments
Closed

Thread Context should collect unique response headers in async actions #19222

pickypg opened this issue Jul 1, 2016 · 2 comments
Labels
:Core/Infra/Core Core issues without another label >enhancement help wanted adoptme high hanging fruit Team:Core/Infra Meta label for core/infra team

Comments

@pickypg
Copy link
Member

pickypg commented Jul 1, 2016

The current implementation of the ThreadContext (#17804) works by having the context passed from thread-to-thread, and the last thread to write to it wins. In the case of async (e.g., scatter/gather) request/response schemes, this won't work if anything unique happens along the way.

This covers all current needs, but it could prove to be very useful to have it actually collect unique async response headers.
@pickypg pickypg mentioned this issue Jul 1, 2016
Merged
@nik9000 nik9000 self-assigned this Apr 19, 2018
@nik9000 nik9000 added the :Core/Infra/Core Core issues without another label label Apr 20, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@nik9000 nik9000 mentioned this issue Apr 20, 2018
Closed
@nik9000 nik9000 removed their assignment Dec 5, 2018
@rjernst rjernst added the Team:Core/Infra Meta label for core/infra team label May 4, 2020
@pugnascotia pugnascotia mentioned this issue Aug 13, 2020
Closed
@rjernst rjernst added the needs:triage Requires assignment of a team area label label Dec 3, 2020
@jaymode jaymode added help wanted adoptme high hanging fruit and removed needs:triage Requires assignment of a team area label labels Dec 14, 2020
@ldematte
Copy link
Contributor

We have mechanisms for handling it, and the issue appears to have been stale for long, closing

@ldematte ldematte closed this as not planned Won't fix, can't repro, duplicate, stale Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Core/Infra/Core Core issues without another label >enhancement help wanted adoptme high hanging fruit Team:Core/Infra Meta label for core/infra team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@clintongormley @nik9000 @rjernst @pickypg @ldematte @jaymode @elasticmachine