From 43dc39ac63fc1a0db799022a47479d5a1b00fa13 Mon Sep 17 00:00:00 2001
From: Yuya Ebihara <ebyhry@gmail.com>
Date: Wed, 14 Feb 2024 22:05:06 +0900
Subject: [PATCH] Remove redundant field

---
 .../ha/module/ClusterStateListenerModule.java        | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/gateway-ha/src/main/java/io/trino/gateway/ha/module/ClusterStateListenerModule.java b/gateway-ha/src/main/java/io/trino/gateway/ha/module/ClusterStateListenerModule.java
index c26ddfdc0..7145b378a 100644
--- a/gateway-ha/src/main/java/io/trino/gateway/ha/module/ClusterStateListenerModule.java
+++ b/gateway-ha/src/main/java/io/trino/gateway/ha/module/ClusterStateListenerModule.java
@@ -13,6 +13,7 @@
  */
 package io.trino.gateway.ha.module;
 
+import com.google.common.collect.ImmutableList;
 import com.google.inject.Provides;
 import com.google.inject.Singleton;
 import io.dropwizard.core.setup.Environment;
@@ -26,13 +27,11 @@
 import io.trino.gateway.ha.router.BackendStateManager;
 import io.trino.gateway.ha.router.RoutingManager;
 
-import java.util.ArrayList;
 import java.util.List;
 
 public class ClusterStateListenerModule
         extends AppModule<HaGatewayConfiguration, Environment>
 {
-    List<TrinoClusterStatsObserver> observers;
     MonitorConfiguration monitorConfig;
 
     public ClusterStateListenerModule(HaGatewayConfiguration config, Environment env)
@@ -51,11 +50,10 @@ public List<TrinoClusterStatsObserver> getClusterStatsObservers(
             RoutingManager mgr,
             BackendStateManager backendStateManager)
     {
-        observers = new ArrayList<>();
-        observers.add(new HealthCheckObserver(mgr));
-        observers.add(new ClusterStatsObserver(backendStateManager));
-
-        return observers;
+        return ImmutableList.<TrinoClusterStatsObserver>builder()
+                .add(new HealthCheckObserver(mgr))
+                .add(new ClusterStatsObserver(backendStateManager))
+                .build();
     }
 
     @Provides