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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
