Remove unnecessary @Inject in HaGatewayProviderModule

andythsu · ebyhr · commit d718b3fc5f29 · 2025-06-06T08:37:16.000+09:00
diff --git a/gateway-ha/src/main/java/io/trino/gateway/ha/module/HaGatewayProviderModule.java b/gateway-ha/src/main/java/io/trino/gateway/ha/module/HaGatewayProviderModule.java
@@ -15,7 +15,6 @@
 
 import com.google.common.collect.ImmutableList;
 import com.google.inject.AbstractModule;
-import com.google.inject.Inject;
 import com.google.inject.Provides;
 import com.google.inject.Singleton;
 import io.airlift.http.client.HttpClient;
@@ -85,7 +84,6 @@ protected void configure()
         jaxrsBinder(binder()).bindInstance(resourceSecurityDynamicFeature);
     }
 
-    @Inject
     public HaGatewayProviderModule(HaGatewayConfiguration configuration)
     {
         this.configuration = requireNonNull(configuration, "configuration is null");

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,6 @@`
`15`	`15`
`16`	`16`	`import com.google.common.collect.ImmutableList;`
`17`	`17`	`import com.google.inject.AbstractModule;`
`18`		`-import com.google.inject.Inject;`
`19`	`18`	`import com.google.inject.Provides;`
`20`	`19`	`import com.google.inject.Singleton;`
`21`	`20`	`import io.airlift.http.client.HttpClient;`
`@@ -85,7 +84,6 @@ protected void configure()`
`85`	`84`	`jaxrsBinder(binder()).bindInstance(resourceSecurityDynamicFeature);`
`86`	`85`	`}`
`87`	`86`
`88`		`- @Inject`
`89`	`87`	`public HaGatewayProviderModule(HaGatewayConfiguration configuration)`
`90`	`88`	`{`
`91`	`89`	`this.configuration = requireNonNull(configuration, "configuration is null");`