search

trinodb / trino-gateway

https://trinodb.github.io/trino-gateway/
Apache License 2.0
122 stars 47 forks source link

Centralize the place to log unhealthy clusters #349

Open andythsu opened 1 month ago

andythsu commented 1 month ago

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:

*