[GitHub] [hadoop] goiri commented on a diff in pull request #4531: HDFS-13274. RBF: Extend RouterRpcClient to use multiple sockets

goiri commented on code in PR #4531: URL: https://github.com/apache/hadoop/pull/4531#discussion_r927072119 ## hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RBFConfigKeys.java: ## @@ -135,6 +135,13 @@ public class RBFConfigKeys

[GitHub] [hadoop] goiri commented on a diff in pull request #4531: HDFS-13274. RBF: Extend RouterRpcClient to use multiple sockets

goiri commented on code in PR #4531: URL: https://github.com/apache/hadoop/pull/4531#discussion_r926048644 ## hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/ConnectionPool.java: ## @@ -210,24 +217,22 @@ public AtomicInteger

[GitHub] [hadoop] goiri commented on a diff in pull request #4531: HDFS-13274. RBF: Extend RouterRpcClient to use multiple sockets

goiri commented on code in PR #4531: URL: https://github.com/apache/hadoop/pull/4531#discussion_r924855528 ## hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/ConnectionPool.java: ## @@ -210,24 +217,22 @@ public AtomicInteger

[GitHub] [hadoop] goiri commented on a diff in pull request #4531: HDFS-13274. RBF: Extend RouterRpcClient to use multiple sockets

goiri commented on code in PR #4531: URL: https://github.com/apache/hadoop/pull/4531#discussion_r923893873 ## hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestConnectionManager.java: ## @@ -304,7 +343,7 @@ private void

[GitHub] [hadoop] goiri commented on a diff in pull request #4531: HDFS-13274. RBF: Extend RouterRpcClient to use multiple sockets

goiri commented on code in PR #4531: URL: https://github.com/apache/hadoop/pull/4531#discussion_r917162075 ## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Client.java: ## @@ -1760,7 +1760,7 @@ UserGroupInformation getTicket() { return ticket;

[GitHub] [hadoop] goiri commented on a diff in pull request #4531: HDFS-13274. RBF: Extend RouterRpcClient to use multiple sockets

goiri commented on code in PR #4531: URL: https://github.com/apache/hadoop/pull/4531#discussion_r915232277 ## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/RPC.java: ## @@ -541,6 +541,28 @@ public static ProtocolProxy getProtocolProxy(Class protocol,