Merge pull request quarkusio#36420 from ahus1/is-36419-blocking-health-checks-should-be-parallelized

jmartisk · web-flow · commit 15a19d214575 · 2023-10-12T12:04:42.000+02:00
Allow parallel execution of blocking health checks
diff --git a/extensions/smallrye-health/runtime/src/main/java/io/quarkus/smallrye/health/runtime/QuarkusAsyncHealthCheckFactory.java b/extensions/smallrye-health/runtime/src/main/java/io/quarkus/smallrye/health/runtime/QuarkusAsyncHealthCheckFactory.java
@@ -29,7 +29,7 @@ public QuarkusAsyncHealthCheckFactory(Vertx vertx) {
     public Uni<HealthCheckResponse> callSync(HealthCheck healthCheck) {
         Uni<HealthCheckResponse> healthCheckResponseUni = super.callSync(healthCheck);
         return BlockingOperationControl.isBlockingAllowed() ? healthCheckResponseUni
-                : healthCheckResponseUni.runSubscriptionOn(MutinyHelper.blockingExecutor(vertx));
+                : healthCheckResponseUni.runSubscriptionOn(MutinyHelper.blockingExecutor(vertx, false));
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public QuarkusAsyncHealthCheckFactory(Vertx vertx) {`
`29`	`29`	`public Uni<HealthCheckResponse> callSync(HealthCheck healthCheck) {`
`30`	`30`	`Uni<HealthCheckResponse> healthCheckResponseUni = super.callSync(healthCheck);`
`31`	`31`	`return BlockingOperationControl.isBlockingAllowed() ? healthCheckResponseUni`
`32`		`- : healthCheckResponseUni.runSubscriptionOn(MutinyHelper.blockingExecutor(vertx));`
	`32`	`+ : healthCheckResponseUni.runSubscriptionOn(MutinyHelper.blockingExecutor(vertx, false));`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`@Override`