From 4915032716535921c3b643139023bec6e2a94643 Mon Sep 17 00:00:00 2001 From: anandheritage Date: Sat, 14 Sep 2024 21:05:06 +0530 Subject: [PATCH] Make the SSL build context as util --- .../org/apache/pinot/query/service/server/QueryServer.java | 6 ------ 1 file changed, 6 deletions(-) diff --git a/pinot-query-runtime/src/main/java/org/apache/pinot/query/service/server/QueryServer.java b/pinot-query-runtime/src/main/java/org/apache/pinot/query/service/server/QueryServer.java index 3dda827ae20..65e4ca7df07 100644 --- a/pinot-query-runtime/src/main/java/org/apache/pinot/query/service/server/QueryServer.java +++ b/pinot-query-runtime/src/main/java/org/apache/pinot/query/service/server/QueryServer.java @@ -21,12 +21,10 @@ import io.grpc.Server; import io.grpc.ServerBuilder; import io.grpc.netty.shaded.io.grpc.netty.NettyServerBuilder; -import io.grpc.netty.shaded.io.netty.handler.ssl.SslContext; import io.grpc.stub.StreamObserver; import java.util.List; import java.util.Map; import java.util.concurrent.CompletableFuture; -import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ExecutorService; import java.util.concurrent.Executors; import java.util.concurrent.TimeUnit; @@ -57,10 +55,6 @@ public class QueryServer extends PinotQueryWorkerGrpc.PinotQueryWorkerImplBase { // TODO: Inbound messages can get quite large because we send the entire stage metadata map in each call. // See https://github.com/apache/pinot/issues/10331 private static final int MAX_INBOUND_MESSAGE_SIZE = 64 * 1024 * 1024; - // the key is the hashCode of the TlsConfig, the value is the SslContext - // We don't use TlsConfig as the map key because the TlsConfig is mutable, which means the hashCode can change. If the - // hashCode changes and the map is resized, the SslContext of the old hashCode will be lost. - private static final Map SERVER_SSL_CONTEXTS_CACHE = new ConcurrentHashMap<>(); private final int _port; private final QueryRunner _queryRunner;