Skip to content

Commit

Permalink
Make ConnectionPoolHealthCheck always healthy
Browse files Browse the repository at this point in the history
See discussion at SEAB-4879
  • Loading branch information
coverbeck committed May 17, 2024
1 parent 0430bbe commit d726867
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,10 @@ protected Result check() throws Exception {

if (activeConnections == maxConnections) {
LOG.info("size: {}, active: {}, idle: {}, calculatedLoad: {}", sizeConnections, activeConnections, idleConnections, loadConnections);
return Result.unhealthy("No database connections available");
} else {
return Result.healthy();
}
// All connections being in use doesn't necessarily mean the container is unhealthy; it's probably just processing many requests.
// See discussion at https://ucsc-cgl.atlassian.net/browse/SEAB-4879. Always return healthy for now; perhaps we'll improve it to
// detect unhealthiness in the future.
return Result.healthy();
}
}

0 comments on commit d726867

Please sign in to comment.