As getClusterSuffixFromConfig() changed we need to change the code for file deletion

Author: sharmaar12

hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AbstractReadOnlyController.java

@@ -19,7 +19,7 @@
 
 import java.io.FileNotFoundException;
 import java.io.IOException;
-import java.nio.charset.StandardCharsets;
+import java.util.Arrays;
 import org.apache.commons.io.IOUtils;
 import org.apache.hadoop.fs.FSDataInputStream;
 import org.apache.hadoop.fs.FileSystem;
@@ -64,9 +64,9 @@ public static void manageActiveClusterIdFile(boolean readOnlyEnabled, MasterFile
         LOG.debug("Global read-only mode is being ENABLED. Deleting active cluster file: {}",
           activeClusterFile);
         try (FSDataInputStream in = fs.open(activeClusterFile)) {
-          String actualClusterFileData = IOUtils.toString(in, StandardCharsets.UTF_8);
-          String expectedClusterFileData = mfs.getActiveClusterSuffix().toString();
-          if (actualClusterFileData.equals(expectedClusterFileData)) {
+          byte[] actualClusterFileData = IOUtils.toByteArray(in);
+          byte[] expectedClusterFileData = mfs.getActiveClusterSuffix().toByteArray();
+          if (Arrays.equals(actualClusterFileData, expectedClusterFileData)) {
             fs.delete(activeClusterFile, false);
             LOG.info("Successfully deleted active cluster file: {}", activeClusterFile);
           } else {