[PR] Add Wait.waitFor() overload for just error msg [accumulo]

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

DomGarguilo opened a new pull request, #6572:
URL: https://github.com/apache/accumulo/pull/6572

   This PR adds a new `waitFor()` overload that accepts the wait condition and an error message. Before this change, if we wanted to add an error message, we needed to use the overload that also specifies the wait duration and sleep values. A lot of the time I think we don't care to change those defaults and just want a good error message. This new overload allows for that.
   
   I did not triage or change any existing usages to be replaced by this new overload since I did not want to assume case by case that we intended to keep those mentioned values default even when they were the same value. This new method is just a convenience for future use.
   
   I also fixed some typos and tightened some of the javadoc comments while I was editing this file.


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

Re: [PR] Add Wait.waitFor() overload for just error msg [accumulo]

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

DomGarguilo merged PR #6572:
URL: https://github.com/apache/accumulo/pull/6572


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