Skip to content

Conversation

ankit--sethi
Copy link
Contributor

@ankit--sethi ankit--sethi commented Oct 8, 2025

As outlined in #126609, TransportSamlLogoutAction and TransportSamlInvalidateSessionAction have a gap in their implementation where execution occurring on the generic thread pool unintentionally forks to a transport_worker thread. This is related to usage of client::get and client::bulk methods within the flow, where client refers to NodeClient that will always execute a given task on a transport worker thread. (see NodeClient.executeLocally and the discussion here.

This change refactors the existing consumers and wraps them such that the follow up work after we have retrieved GetResponse/BulkResponse is re-directed back to the generic thread pool, i.e. after client has done its job retrieving these responses.

elasticsearchmachine and others added 3 commits October 8, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants