mhansonp commented on code in PR #7608: URL: https://github.com/apache/geode/pull/7608#discussion_r857852438
########## geode-cq/src/distributedTest/java/org/apache/geode/internal/cache/tier/sockets/DurableClientTestBase.java: ########## @@ -216,33 +208,35 @@ void verifyDurableClientPresence(int durableClientTimeout, String durableClientI }); } + void waitUntilHARegionQueueSizeIsZero(VM serverVM) { + serverVM.invoke(() -> await().atMost(60, SECONDS) + .until(() -> getClientProxy().getHARegionQueue().size() == 0)); + } + public void closeDurableClient() { durableClientVM.invoke((SerializableRunnableIF) CacheServerTestUtil::closeCache); } public void disconnectDurableClient(boolean keepAlive) { - printClientProxyState("Before"); + // printClientProxyState("Before"); durableClientVM.invoke("close durable client cache", () -> CacheServerTestUtil.closeCache(keepAlive)); - await() - .until(CacheServerTestUtil::getCache, nullValue()); - printClientProxyState("after"); + await().until(CacheServerTestUtil::getCache, nullValue()); + // printClientProxyState("after"); } private void printClientProxyState(String st) { - CacheSerializableRunnable s = - new CacheSerializableRunnable("Logging CCCP and ServerConnection state") { - @Override - public void run2() throws CacheException { - // TODO Auto-generated method stub - CacheServerTestUtil.getCache().getLogger() - .info(st + " CCP states: " + getAllClientProxyState()); - CacheServerTestUtil.getCache().getLogger().info(st + " CHM states: " - + printMap( - ClientHealthMonitor.getInstance().getConnectedClients(null))); - } - }; - server1VM.invoke(s); + server1VM.invoke("Logging CCCP and ServerConnection state", new CacheSerializableRunnable() { + @Override + public void run2() throws CacheException { + // TODO Auto-generated method stub Review Comment: removed -- 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-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org