[GitHub] [accumulo] DomGarguilo commented on a diff in pull request #3292: fixes #3252 support tablets w/ and w/o location in tablet location cache

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo commented on code in PR #3292:
URL: https://github.com/apache/accumulo/pull/3292#discussion_r1171810234


##########
core/src/main/java/org/apache/accumulo/core/clientImpl/ThriftScanner.java:
##########
@@ -299,10 +305,52 @@ public ScanState(ClientContext context, TableId tableId, Authorizations authoriz
     }
   }
 
-  public static class ScanTimedOutException extends IOException {
+  static <T> Optional<T> waitUntil(Supplier<Optional<T>> condition, Duration maxWaitTime,

Review Comment:
   It does seem like there could be some consolidation with these however I'm not sure what the best way to consolidate them would be.



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