ddanielr commented on issue #4594: URL: https://github.com/apache/accumulo/issues/4594#issuecomment-2136088991
> An IT already exists for this, see https://github.com/apache/accumulo/blob/2.1/test/src/main/java/org/apache/accumulo/test/ScanServerMetadataEntriesIT.java#L140 That IT only tests the scan server ref expiration removal. Looking at the IT config, the scan refs expire after 5 seconds https://github.com/apache/accumulo/blob/04f5aaf7c27695b6d4ac0383bf79ef74ed3e54a8/test/src/main/java/org/apache/accumulo/test/ScanServerMetadataEntriesIT.java#L82 If that test is modified to shutdown the scan server vs wait for the refs to expire then it runs for almost 10min until the manager cleans up the scan server refs. ``` // Trigger Shutdown of scan server. getCluster().getClusterControl().stop(ServerType.SCAN_SERVER); // close happens asynchronously. Let the test fail by timeout while (ctx.getAmple().getScanServerFileReferences().findAny().isPresent()) { log.info("Scan Server Refs are still present"); log.info("Refs: {}", ctx.getAmple().getScanServerFileReferences().collect(Collectors.toList())); Thread.sleep(1000); } ``` That can be verified by reducing the amount of time the manager waits before cleaning up references and the test will run until the new specified interval. https://github.com/apache/accumulo/blob/04f5aaf7c27695b6d4ac0383bf79ef74ed3e54a8/server/manager/src/main/java/org/apache/accumulo/manager/Manager.java#L1253-L1254 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
