[GitHub] keith-turner commented on a change in pull request #299: ACCUMULO-4702 Removed Beta Guava dependencies

[gi...@git.apache.org]

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:
users@infra.apache.org


With regards,
Apache Git Services