[GitHub] [hadoop] 9uapaw commented on a diff in pull request #3259: HADOOP-15327. Upgrade MR ShuffleHandler to use Netty4

2022-11-09 Thread GitBox
9uapaw commented on code in PR #3259: URL: https://github.com/apache/hadoop/pull/3259#discussion_r1018349426 ## hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/Fetcher.java: ## @@ -72,19 +73,21

[GitHub] [hadoop] 9uapaw commented on a diff in pull request #3259: HADOOP-15327. Upgrade MR ShuffleHandler to use Netty4

2022-11-09 Thread GitBox
9uapaw commented on code in PR #3259: URL: https://github.com/apache/hadoop/pull/3259#discussion_r1018349080 ## hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java: ## @@ -182,19 +184,29 @@

[GitHub] [hadoop] 9uapaw commented on a diff in pull request #3259: HADOOP-15327. Upgrade MR ShuffleHandler to use Netty4

2022-11-09 Thread GitBox
9uapaw commented on code in PR #3259: URL: https://github.com/apache/hadoop/pull/3259#discussion_r1018347131 ## hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java: ## @@ -291,36 +302,86 @@

[GitHub] [hadoop] 9uapaw commented on a diff in pull request #3259: HADOOP-15327. Upgrade MR ShuffleHandler to use Netty4

2022-11-09 Thread GitBox
9uapaw commented on code in PR #3259: URL: https://github.com/apache/hadoop/pull/3259#discussion_r1018346414 ## hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java: ## @@ -904,65 +990,84 @@

[GitHub] [hadoop] 9uapaw commented on a diff in pull request #3259: HADOOP-15327. Upgrade MR ShuffleHandler to use Netty4

2022-11-09 Thread GitBox
9uapaw commented on code in PR #3259: URL: https://github.com/apache/hadoop/pull/3259#discussion_r1018309405 ## hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/test/resources/log4j.properties: ## @@ -17,3 +17,5 @@ log4j.threshold=ALL

[GitHub] [hadoop] 9uapaw commented on a diff in pull request #3259: HADOOP-15327. Upgrade MR ShuffleHandler to use Netty4

2022-04-05 Thread GitBox
9uapaw commented on code in PR #3259: URL: https://github.com/apache/hadoop/pull/3259#discussion_r842824343 ## hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java: ## @@ -1081,6 +1200,9 @@