bhattmanish98 commented on code in PR #7669:
URL: https://github.com/apache/hadoop/pull/7669#discussion_r2465902645


##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/constants/ConfigurationKeys.java:
##########
@@ -459,10 +459,43 @@ public static String containerProperty(String property, 
String fsName, String ac
   public static final String FS_AZURE_BLOB_DIR_RENAME_MAX_THREAD = 
"fs.azure.blob.dir.rename.max.thread";
   /**Maximum number of thread per blob-delete orchestration: {@value}*/
   public static final String FS_AZURE_BLOB_DIR_DELETE_MAX_THREAD = 
"fs.azure.blob.dir.delete.max.thread";
+  /**
+   * Configuration key for the keep-alive time for the write thread pool.
+   * This value specifies the amount of time that threads in the write thread 
pool
+   * will remain idle before being terminated.
+   * Value: {@value}.
+   */
+  public static final String FS_AZURE_WRITE_THREADPOOL_KEEP_ALIVE_TIME = 
"fs.azure.write.threadpool.keep.alive.time";

Review Comment:
   It is good if we rename this variable to 
FS_AZURE_WRITE_THREADPOOL_KEEP_ALIVE_TIME_SECONDS = 
"fs.azure.write.threadpool.keep.alive.time.seconds". This will help us to know 
in future in which unit this time is.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to