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