Skip to content

chore: remove async_executor param in ToolInvoker #9548

Closed
@Amnah199

Description

@Amnah199
Contributor

With the update introduced in PR #9530, the async_executor parameter has been deprecated in ToolInvoker. As a result:

  • The async_executor argument should now be removed from the __init__ method.

  • The internal flag self._owns_executor is no longer necessary and should also be removed as part of this cleanup.

This aligns with the new approach of using the max_workers parameter to manage parallel execution via a default ThreadPoolExecutor.

Checklist

Tasks

Activity

added this to the 2.16.0 milestone on Jun 24, 2025
changed the title [-]chore: deprecate `async_executor` param in ToolInvoker[/-] [+]chore: remove `async_executor` param in ToolInvoker[/+] on Jun 24, 2025
added
P2Medium priority, add to the next sprint if no P1 available
P1High priority, add to the next sprint
and removed
P2Medium priority, add to the next sprint if no P1 available
on Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

P1High priority, add to the next sprintbreaking change

Type

No type

Projects

No projects

Relationships

None yet

    Participants

    @julian-risch@Amnah199

    Issue actions

      chore: remove `async_executor` param in ToolInvoker · Issue #9548 · deepset-ai/haystack