Mailing-List: contact core-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: core-dev@hadoop.apache.org
Message-ID: <1843517726.1228023464271.JavaMail.jira@brutus>
Date: Sat, 29 Nov 2008 21:37:44 -0800 (PST)
From: "Chris Douglas (JIRA)" <jira@apache.org>
To: core-dev@hadoop.apache.org
Subject: [jira] Updated: (HADOOP-4732)  connectionTimeout and readTimeout
 are passed in wrong order in ReduceTask.java
In-Reply-To: <624968479.1227697964252.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/HADOOP-4732?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris Douglas updated HADOOP-4732:
----------------------------------

      Resolution: Fixed
    Hadoop Flags: [Reviewed]
          Status: Resolved  (was: Patch Available)

+1 Good catch.

I just committed this. Thanks, Amareshwari

>  connectionTimeout and readTimeout are passed in wrong order in ReduceTask.java
> -------------------------------------------------------------------------------
>
>                 Key: HADOOP-4732
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4732
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: mapred
>            Reporter: Amareshwari Sriramadasu
>            Assignee: Amareshwari Sriramadasu
>            Priority: Minor
>             Fix For: 0.20.0
>
>         Attachments: patch-4732.txt
>
>
> In ReduceTask.java, The parameters for method calls for getInputStream(URLConnection connection,  int connectionTimeout,  int readTimeout) should be in that order. Now it is
> {code}
>         InputStream input = getInputStream(connection, DEFAULT_READ_TIMEOUT, 
>                                            STALLED_COPY_TIMEOUT);
> {code}
> It works now because both DEFAULT_READ_TIMEOUT and STALLED_COPY_TIMEOUT have the same value.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.