Re: Please use Awaitilty instead of Thread.sleep

2018-03-16 Thread Jens Deppe
My bad. Thanks for pointing that out Kirk. On Fri, Mar 16, 2018 at 11:22 AM, Kirk Lund 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. > ClusterConfigLocatorRestartDU

Please use Awaitilty instead of Thread.sleep

2018-03-16 Thread Kirk Lund
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(1); Please don't d