The implementation of HealthCheck.performAsyncHealthCheck is not async

Description

Contrary to its name, DefaultHealthCheckService.performAsyncHealthCheck performs all health checks one after another in a blocking way.

Health checks could be executed in parallel and the final result could be gathered in a non-blocking way.

Acceptance / Success Criteria

Health checks could be executed in parallel and the final result could be gathered in a non-blocking way.

Lucidchart Diagrams

Activity

Stefan Wachter October 8, 2021 at 12:36 PM

Stefan Wachter September 29, 2021 at 9:50 AM
Edited

Sure, will target foundation-2021.

On second thought though, my recent work on the passive health checks targets release-28.x. That work contains some changes regarding the collection of health check results. I think there will be some merge conflicts when this work targets foundation-2021.

In summary, I would prefer to base this work on the passive health checks and target release-28.x.

Benjamin Reed September 28, 2021 at 6:04 PM

Can this be done in Foundation 2021 as well, do you think?

Fixed

Details

Assignee

Reporter

Story Points

Sprint

Fix versions

Priority

PagerDuty

Created September 27, 2021 at 8:46 AM
Updated April 27, 2022 at 4:38 PM
Resolved October 19, 2021 at 3:10 PM