[ 
https://issues.apache.org/jira/browse/TEZ-988?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13950514#comment-13950514
 ] 

Mohammad Kamrul Islam commented on TEZ-988:
-------------------------------------------

Thanks [~rajesh.balamohan] for uploading the patch quickly.

> url.append("&keepAlive=").append(keepAlive)

I'm not sure if the value could be set through mapred-site.xml. Your comments 
said that it could be turned off through mapred-site.xml. Not sure it can be 
turned on as well.  If it can be turned on through mapred-site.xml, I think we 
could be overloading by tez default "false".


> http.maxConnections needs to be configurable in Tez Fetcher & read from 
> errorstream to make the connection reusable
> -------------------------------------------------------------------------------------------------------------------
>
>                 Key: TEZ-988
>                 URL: https://issues.apache.org/jira/browse/TEZ-988
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.4.0
>            Reporter: Rajesh Balamohan
>            Assignee: Rajesh Balamohan
>         Attachments: TEZ-988-v1.patch, TEZ-988-v2.patch
>
>
> 1. Currently http.maxConnections is set to 5 (default).  Make this 
> configurable in Fetcher.java.  This will help in running larger queries
> 2. ErrorStream has to be read completely in order to make the connection 
> reusable (when keepAlive is enabled).  Currently, we do not read error stream.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to