Fix minor naming conflict after rebase with flight server pr. boundAddress() -> getBoundAddress().

Signed-off-by: Finn Carroll <[email protected]>

Author: finnegancarroll

plugins/arrow-flight-rpc/src/main/java/org/opensearch/arrow/flight/bootstrap/FlightService.java

@@ -134,6 +134,7 @@ public StreamManager getStreamManager() {
      * Retrieves the bound address of the FlightService.
      * @return The BoundTransportAddress instance.
      */
+    @Override
     public BoundTransportAddress getBoundAddress() {
         return serverComponents.getBoundAddress();
     }

plugins/transport-grpc/src/main/java/org/opensearch/transport/grpc/Netty4GrpcServerTransport.java

@@ -156,7 +156,7 @@ public Netty4GrpcServerTransport(Settings settings, List<BindableService> servic
 
     // public for tests
     @Override
-    public BoundTransportAddress boundAddress() {
+    public BoundTransportAddress getBoundAddress() {
         return this.boundAddress;
     }
 

plugins/transport-grpc/src/test/java/org/opensearch/transport/grpc/Netty4GrpcServerTransportIT.java

@@ -30,7 +30,7 @@ public class Netty4GrpcServerTransportIT extends OpenSearchIntegTestCase {
     private TransportAddress randomNetty4GrpcServerTransportAddr() {
         List<TransportAddress> addresses = new ArrayList<>();
         for (Netty4GrpcServerTransport transport : internalCluster().getInstances(Netty4GrpcServerTransport.class)) {
-            TransportAddress tAddr = new TransportAddress(transport.boundAddress().publishAddress().address());
+            TransportAddress tAddr = new TransportAddress(transport.getBoundAddress().publishAddress().address());
             addresses.add(tAddr);
         }
         return randomFrom(addresses);

plugins/transport-grpc/src/test/java/org/opensearch/transport/grpc/Netty4GrpcServerTransportTests.java

@@ -36,16 +36,16 @@ public void setup() {
     public void testGrpcTransportStartStop() {
         try (Netty4GrpcServerTransport transport = new Netty4GrpcServerTransport(createSettings(), services, networkService)) {
             transport.start();
-            MatcherAssert.assertThat(transport.boundAddress().boundAddresses(), not(emptyArray()));
-            assertNotNull(transport.boundAddress().publishAddress().address());
+            MatcherAssert.assertThat(transport.getBoundAddress().boundAddresses(), not(emptyArray()));
+            assertNotNull(transport.getBoundAddress().publishAddress().address());
             transport.stop();
         }
     }
 
     public void testGrpcTransportHealthcheck() {
         try (Netty4GrpcServerTransport transport = new Netty4GrpcServerTransport(createSettings(), services, networkService)) {
             transport.start();
-            final TransportAddress remoteAddress = randomFrom(transport.boundAddress().boundAddresses());
+            final TransportAddress remoteAddress = randomFrom(transport.getBoundAddress().boundAddresses());
             try (NettyGrpcClient client = new NettyGrpcClient.Builder().setAddress(remoteAddress).build()) {
                 assertEquals(client.checkHealth(), HealthCheckResponse.ServingStatus.SERVING);
             }

plugins/transport-grpc/src/test/java/org/opensearch/transport/grpc/SecureNetty4GrpcServerTransportTests.java

@@ -132,8 +132,8 @@ public void testGrpcSecureTransportStartStop() {
             )
         ) {
             transport.start();
-            assertTrue(transport.boundAddress().boundAddresses().length > 0);
-            assertNotNull(transport.boundAddress().publishAddress().address());
+            assertTrue(transport.getBoundAddress().boundAddresses().length > 0);
+            assertNotNull(transport.getBoundAddress().publishAddress().address());
             transport.stop();
         } catch (Exception e) {
             throw new RuntimeException(e);
@@ -150,9 +150,9 @@ public void testGrpcSecureTransportHealthcheck() {
             )
         ) {
             transport.start();
-            assertTrue(transport.boundAddress().boundAddresses().length > 0);
-            assertNotNull(transport.boundAddress().publishAddress().address());
-            final TransportAddress remoteAddress = randomFrom(transport.boundAddress().boundAddresses());
+            assertTrue(transport.getBoundAddress().boundAddresses().length > 0);
+            assertNotNull(transport.getBoundAddress().publishAddress().address());
+            final TransportAddress remoteAddress = randomFrom(transport.getBoundAddress().boundAddresses());
             try (
                 NettyGrpcClient client = new NettyGrpcClient.Builder().setAddress(remoteAddress)
                     .setSecureSettingsProvider(settingsProvider)

server/src/main/java/org/opensearch/plugins/NetworkPlugin.java

@@ -95,7 +95,7 @@ abstract class AuxTransport extends AbstractLifecycleComponent {
         );
 
         // public for tests
-        public abstract BoundTransportAddress boundAddress();
+        public abstract BoundTransportAddress getBoundAddress();
     }
 
     /**