kabhishek4 commented on code in PR #6887: URL: https://github.com/apache/hbase/pull/6887#discussion_r2113654954
########## hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java: ########## @@ -382,4 +393,68 @@ public void stop() { public void logFileSystemState(Logger log) throws IOException { CommonFSUtils.logFileSystemState(fs, rootdir, log); } + + private void negotiateActiveClusterSuffixFile(long wait) throws IOException { + String suffix_file_name = HConstants.ACTIVE_CLUSTER_SUFFIX_FILE_NAME; + boolean activeClusterSuffixFileExists = + FSUtils.checkFileExistsInHbaseRootDir(fs, rootdir, suffix_file_name, wait); Review Comment: Yes. I think it is possible to use exception types coming from getActiveClusterSuffix method and update the logic here to do it in one step. I will try out the changes and update the patch. -- 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