Skip to content

Conversation

nicktindall
Copy link
Contributor

Resolves: ES-13138

: "was last called [" + TimeValue.timeValueMillis(timeSinceLastRerouteMillis) + "] ago",
nodeSummary(lastSetOfHotSpottedNodes),
state.nodes().size()
writeLoadConstraintSettings.getQueueLatencyThreshold()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Found an actual issue 🎉

@nicktindall nicktindall 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) labels Oct 13, 2025
@nicktindall nicktindall marked this pull request as ready for review October 13, 2025 04:08
@nicktindall nicktindall requested a review from ywangd October 13, 2025 04:08
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Oct 13, 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

Comment on lines +121 to +123
// Clear simulated hot-spotting
MockTransportService.getInstance(dataNodeOne).clearAllRules();
MockTransportService.getInstance(dataNodeTwo).clearAllRules();
Copy link
Member

Choose a reason for hiding this comment

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

Is it possible to test for the message of Not calling reroute because we called reroute [{}] ago and there are no new hot spots as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's on line 95 I think, or did you mean something else?

Copy link
Member

Choose a reason for hiding this comment

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

Ah I was blind 😮‍💨

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.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants