From 854af9cee167749b9c7470b171aa7e499f4c7f80 Mon Sep 17 00:00:00 2001 From: Akira Ajisaka Date: Tue, 23 Aug 2016 17:04:55 +0900 Subject: [PATCH] MAPREDUCE-6587. Remove unused params in connection-related methods of Fetcher. Contributed by Yiqun Lin. (cherry picked from commit 8cc4a67059e37b2083cd5468b35a64a403a3e3ae) (cherry picked from commit 41fe85f8688e6edd2a94ffd831d776e7c7a3614f) --- .../apache/hadoop/mapreduce/task/reduce/Fetcher.java | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/Fetcher.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/Fetcher.java index d8dd7b5f32b..be2f84f0175 100644 --- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/Fetcher.java +++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/task/reduce/Fetcher.java @@ -263,7 +263,7 @@ class Fetcher extends Thread { DataInputStream input = null; try { - setupConnectionsWithRetry(host, remaining, url); + setupConnectionsWithRetry(url); if (stopped) { abortConnect(host, remaining); } else { @@ -374,9 +374,8 @@ class Fetcher extends Thread { } } - private void setupConnectionsWithRetry(MapHost host, - Set remaining, URL url) throws IOException { - openConnectionWithRetry(host, remaining, url); + private void setupConnectionsWithRetry(URL url) throws IOException { + openConnectionWithRetry(url); if (stopped) { return; } @@ -396,8 +395,7 @@ class Fetcher extends Thread { verifyConnection(url, msgToEncode, encHash); } - private void openConnectionWithRetry(MapHost host, - Set remaining, URL url) throws IOException { + private void openConnectionWithRetry(URL url) throws IOException { long startTime = Time.monotonicNow(); boolean shouldWait = true; while (shouldWait) {