[GitHub] [accumulo] ctubbsii commented on pull request #2990: Block use of fate in Manager until fate is ready

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

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

   > I tested these changes and the new error is being printed just as much but with more information:
   
   I was hoping it might occur less often, because of the way it blocks, it might prevent noisy NPEs from retries.
   
   I think the main concern is: do these make progress? Or do they block indefinitely?
   
   If they make progress, it's probably fine. We can tune this a bit further... maybe instead of logging and waiting, we just wait for a certain amount of time, then log and throw an exception only if it times out, and fails to make progress. Setting a timeout is easy to do with the CountDownLatch. How much time does the Manager need, typically?


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