steveloughran commented on a change in pull request #3042:
URL: https://github.com/apache/hadoop/pull/3042#discussion_r637533874



##########
File path: 
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java
##########
@@ -4004,12 +4004,14 @@ public void cleanUp() {
      * Background action to act on references being removed.
      */
     private static class StatisticsDataReferenceCleaner implements Runnable {
+      private static int REF_QUEUE_POLL_TIMEOUT = 100;

Review comment:
       that's going to be waking every 100 milliseconds, demanding cpu time etc 
etc. If there has to be a timeout, it needs to be something less disruptive, 
like 100 seconds.
   
   What would happen if that was the case?




-- 
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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

Reply via email to