Run CompositeHealthIndicator in parallel - #15770
Closed
RikEnde wants to merge 4 commits into
Closed
Conversation
|
@RikEnde Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
|
@RikEnde Thank you for signing the Contributor License Agreement! |
spring-projects-issues
added
the
status: waiting-for-triage
RikEnde
changed the title
philwebb
added
the
for: team-attention
snicoll
changed the title
Author
|
I'm pretty sure my commit didn't break the build or cause that timeout... |
Member
|
@RikEnde you are right, your change did not break the build. We have a task execution infrastructure now in place and we'd rather use that if it's available rather than using |
snicoll
added
status: declined
Author
|
Fair enough. Are you referring to the ConcurrentTaskExecutor, or something new? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See #2652
Problem:
The CompositeHealthIndicator polls all the HealthIndicators in the registry sequentially. I have a number of HealtIndicator components that poll remote systems and report on latency and status, and the latency really adds up, especially if there are time-outs involved.
Solution:
This PR polls the HealthIndicators concurrently. It preserves the original ordering, and outputs the same as the old implementation, except the runtime is proportional to that of the slowest component, rather than the aggregate of all components. The impact of this change should be very small, unless someone's code absolutely depends on HealthIndicators being polled sequentially but in unknown order.