2005hithlj commented on code in PR #4811: URL: https://github.com/apache/hbase/pull/4811#discussion_r991271572
########## hbase-server/src/test/java/org/apache/hadoop/hbase/master/cleaner/TestLogsCleaner.java: ########## @@ -314,4 +318,24 @@ public ZKWatcher getZooKeeper() { return null; } } + + private static class FSTableDescriptorsTest extends FSTableDescriptors { Review Comment: > Do we really need this class? Just use Mockito to mock the return value of exists method is enough? Yes, it can be done. And the new implementation is in the new commit. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org