keith-turner commented on a change in pull request #299: ACCUMULO-4702 Removed 
Beta Guava dependencies
URL: https://github.com/apache/accumulo/pull/299#discussion_r138989375
 
 

 ##########
 File path: fate/src/main/java/org/apache/accumulo/fate/util/UtilWaitThread.java
 ##########
 @@ -29,4 +33,36 @@ public static void sleep(long millis) {
       log.error("{}", e.getMessage(), e);
     }
   }
+
+  /**
+   * Copied from Guava release 23. The Uniterruptibles class was annotated as 
Beta by Google, therefore unstable to use. The following javadoc was copied from
+   * com.google.common.util.concurrent.Uninterruptibles:
+   *
+   * Utilities for treating interruptible operations as uninterruptible. In 
all cases, if a thread is interrupted during such a call, the call continues to
+   * block until the result is available or the timeout elapses, and only then 
re-interrupts the thread.
+   *
+   * @author Anthony Zana
+   * @since 10.0
+   */
+  public static void sleepUninterruptibly(long sleepFor, TimeUnit unit) {
 
 Review comment:
   It may be nice to put all copied guava code into one package.  Not sure if 
this is possible.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to