trino-gateway icon indicating copy to clipboard operation
trino-gateway copied to clipboard

Published 20 hours ago • verified publisher icon trinodb

Centralize the place to log unhealthy clusters

Open andythsu opened this issue 9 months ago • 0 comments

Description

As different routing rules are implemented, routingManager.updateBackEndStats method could be overridden. An example is https://github.com/trinodb/trino-gateway/blob/c052bb237450d7fc2f2f7af1b06f7fc8f4cdee48/gateway-ha/src/main/java/io/trino/gateway/ha/router/QueryCountBasedRouter.java#L238-L241

Therefore, it will be more useful to log unhealthy clusters at ActiveClusterMonitor level instead of RoutingManager level

Additional context and related issues

Release notes

(X) This is not user-visible or is docs only, and no release notes are required. ( ) Release notes are required. Please propose a release note for me. ( ) Release notes are required, with the following suggested text:

*

andythsu avatar May 20 '24 21:05 andythsu