My bad. Thanks for pointing that out Kirk.

On Fri, Mar 16, 2018 at 11:22 AM, Kirk Lund <kl...@apache.org> wrote:

> I can't believe we're still checking in new tests with Thread.sleep of 10
> seconds instead of using Awaitility.
>
> org.apache.geode.management.internal.configuration.
> ClusterConfigLocatorRestartDUnitTest
>
> In both test methods:
>
>     // Wait some time to reconnect
>     Thread.sleep(10000);
>
> Please don't do this!
>

Reply via email to