hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Xiao Chen (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-11100) Support to configure ftpClient.setControlKeepAliveTimeout
Date Fri, 12 Oct 2018 18:08:00 GMT

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

Xiao Chen commented on HADOOP-11100:
------------------------------------

Thanks [~adam.antal] for the good work here, and [~knanasi] for the review.

Overall LGTM. 2 additional comments:
 * We should add a core-default.xml entry for the new config. \{{TestCommonConfigurationFields}}
usually catches this, but ftp is kind of a special case so missed here. Once the config is
added to core-default.xml, you'd need to update \{{TestCommonConfigurationFields}} to make
the test pass.
 * Adding a timeout is a nice improvement. Strictly speaking, setting the default changes
the current behavior. In general we try not to change the default behavior, for compatibility
reasons. Only when something is obviously wrong, to the point which we consider it a bug,
would we change the default behavior. I'd suggest we set the default to 0 here, to retain
existing behavior (since FTPFileSystem has been there for a long time and is marked as Public
Stable).

> Support to configure  ftpClient.setControlKeepAliveTimeout 
> -----------------------------------------------------------
>
>                 Key: HADOOP-11100
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11100
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>    Affects Versions: 2.3.0
>            Reporter: Krishnamoorthy Dharmalingam
>            Assignee: Adam Antal
>            Priority: Minor
>         Attachments: HADOOP-11100.002.patch, HADOOP-11100.003.patch, HDFS-11000.001.patch
>
>
> In FTPFilesystem or Configuration, timeout is not possible to configure.
> It is very straight forward to configure, in FTPFilesystem.connect() method.
>  ftpClient.setControlKeepAliveTimeout
> Like
> private FTPClient connect() throws IOException {
> ...
> String timeout = conf.get("fs.ftp.timeout." + host);
> ...
>  ftpClient.setControlKeepAliveTimeout(new Integer(300));
> ....
> }



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org


Mime
View raw message