Skip to content

Conversation

@DiannaHohensee
Copy link
Contributor

@DiannaHohensee DiannaHohensee commented Jul 11, 2025

Cannot expect thread pool task to have
completed because the test only asserts
that the task has started execution

Closes #131093


Specifically, this line waits for the task to start (that lambda is an override for all runnable tasks submitted to the thread pool ). Not finish and be cleared from the ongoing tasks tracking list.

Cannot expect thread pool task to have
completed because the test only asserts
that the task has started execution

Closes elastic#131093
@DiannaHohensee DiannaHohensee self-assigned this Jul 11, 2025
@DiannaHohensee DiannaHohensee added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) Team:Distributed Coordination Meta label for Distributed Coordination team labels Jul 11, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination)

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DiannaHohensee DiannaHohensee merged commit 8ba2ea9 into elastic:main Jul 14, 2025
6 of 8 checks passed
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jul 17, 2025
Cannot expect thread pool task to have
completed because the test only asserts
that the task has started execution

Closes elastic#131093
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jul 17, 2025
Cannot expect thread pool task to have
completed because the test only asserts
that the task has started execution

Closes elastic#131093
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) Team:Distributed Coordination Meta label for Distributed Coordination team >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] TaskExecutionTimeTrackingEsThreadPoolExecutorTests testMaxQueueLatency failing

3 participants