[GitHub] [accumulo] ctubbsii commented on pull request #3121: Replace UtilWaitThread.sleepUninterruptibly with guava equivalent

[GitBox <gi...@apache.org>]

ctubbsii commented on PR #3121:
URL: https://github.com/apache/accumulo/pull/3121#issuecomment-1352029056

   I don't think we have a plan to remove Guava entirely. We have tried to avoid the problematic beta methods, certainly. It's also good to reduce our dependency on any external library, Guava is not an exception to that. The `modernizer-maven-plugin` helps identify some of those things. For non-beta methods that we expect to be stable, and for which we expect a minimum Guava version to be required, I don't think it's a problem to use the utility that Guava provides. We'll need to fix the dependency check first before this is merged (just add the guava dependency explicitly to the modules that didn't have it as a compile time dependency before, but now do).


-- 
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: notifications-unsubscribe@accumulo.apache.org

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