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

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

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

   > It definitely makes sense to me to first try and use native Java code/utilities when possible, such as lambdas instead of Guavas iterable utilities. But if something doesn't exist I think it would be fine to use the utility in Guava or something like a Commons library so we don't have to re-invent the wheel, assuming the plan isn't to actually remove the dependency entirely at some point.
   
   I agree with these points.
   
   @EdColeman I don't know of/can't find any native java replacements for this. That's not to say one does not exist but I haven't been able to find anything yet.
   
   @dlmarion As for removing guava, I am not sure either. My understanding is that when possible, we should avoid using guava for a built in equivalent, not sure if we have plans to completely remove it as a dependency though.


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