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

ASF GitHub Bot commented on HADOOP-18845:
-----------------------------------------

mukund-thakur commented on code in PR #5948:
URL: https://github.com/apache/hadoop/pull/5948#discussion_r1302108970


##########
hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/Constants.java:
##########
@@ -156,10 +156,16 @@ private Constants() {
   public static final String MAXIMUM_CONNECTIONS = "fs.s3a.connection.maximum";
   public static final int DEFAULT_MAXIMUM_CONNECTIONS = 96;
 
-  // Expiration time of s3 http connection from the connection pool.
-  // See {@code com.amazonaws.ClientConfiguration#setConnectionTTL}
+  /**
+   * Configuration option to configure expiration time of
+   * s3 http connection from the connection pool in milliseconds: {@value}.
+   */
   public static final String CONNECTION_TTL = "fs.s3a.connection.ttl";
-  public static final long DEFAULT_CONNECTION_TTL = 
ClientConfiguration.DEFAULT_CONNECTION_TTL;
+
+  /**
+   * Default value for {@value CONNECTION_TTL}: {@value}.

Review Comment:
   It will convert it to the actual value so I think it is good 
https://javadoc.io/doc/org.apache.hadoop/hadoop-aws/latest/index.html 
   





> Add ability to configure ConnectionTTL of http connections while creating S3 
> Client.
> ------------------------------------------------------------------------------------
>
>                 Key: HADOOP-18845
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18845
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 3.3.6
>            Reporter: Mukund Thakur
>            Assignee: Mukund Thakur
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.3.9
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to