[GitHub] [accumulo] DomGarguilo opened a new pull request, #3185: Fix usage of Wait.waitFor() in ITs

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

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

   After reviewing #3177 I realized that some of the spots that `Wait.waitFor()` is used in the tests does not check the value of the returned `boolean`. This PR asserts that the cases that were not being checked, now are.


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

[GitHub] [accumulo] DomGarguilo merged pull request #3185: Fix usage of Wait.waitFor() in ITs

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

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


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

[GitHub] [accumulo] ctubbsii commented on pull request #3185: Fix usage of Wait.waitFor() in ITs

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

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

   This was originally merged to main first, but then backported into 2.1 for 2.1.1 in #3186.


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