HBASE-29207 The backup system table should be considered a system table

Author: Ray Mattingly

hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/BackupRestoreConstants.java

@@ -18,6 +18,7 @@
 package org.apache.hadoop.hbase.backup;
 
 import org.apache.hadoop.hbase.HConstants;
+import org.apache.hadoop.hbase.NamespaceDescriptor;
 import org.apache.yetus.audience.InterfaceAudience;
 
 /**
@@ -29,7 +30,7 @@ public interface BackupRestoreConstants {
    * Backup/Restore constants
    */
   String BACKUP_SYSTEM_TABLE_NAME_KEY = "hbase.backup.system.table.name";
-  String BACKUP_SYSTEM_TABLE_NAME_DEFAULT = "backup:system";
+  String BACKUP_SYSTEM_TABLE_NAME_DEFAULT = NamespaceDescriptor.BACKUP_NAMESPACE_NAME_STR + ":system";
 
   String BACKUP_SYSTEM_TTL_KEY = "hbase.backup.system.ttl";
 

hbase-common/src/main/java/org/apache/hadoop/hbase/NamespaceDescriptor.java

@@ -41,6 +41,9 @@ public class NamespaceDescriptor {
   /** Default namespace name. */
   public static final byte[] DEFAULT_NAMESPACE_NAME = Bytes.toBytes("default");
   public static final String DEFAULT_NAMESPACE_NAME_STR = Bytes.toString(DEFAULT_NAMESPACE_NAME);
+  /** Backup namespace name. */
+  public static final byte[] BACKUP_NAMESPACE_NAME = Bytes.toBytes("backup");
+  public static final String BACKUP_NAMESPACE_NAME_STR = Bytes.toString(BACKUP_NAMESPACE_NAME);
 
   public static final NamespaceDescriptor DEFAULT_NAMESPACE =
     NamespaceDescriptor.create(DEFAULT_NAMESPACE_NAME_STR).build();

hbase-common/src/main/java/org/apache/hadoop/hbase/TableName.java

@@ -301,6 +301,10 @@ private TableName(ByteBuffer namespace, ByteBuffer qualifier) throws IllegalArgu
         this.namespace = NamespaceDescriptor.SYSTEM_NAMESPACE_NAME;
         this.namespaceAsString = NamespaceDescriptor.SYSTEM_NAMESPACE_NAME_STR;
         this.systemTable = true;
+      } else if (Bytes.equals(NamespaceDescriptor.BACKUP_NAMESPACE_NAME, namespace)) {
+        this.namespace = NamespaceDescriptor.BACKUP_NAMESPACE_NAME;
+        this.namespaceAsString = NamespaceDescriptor.BACKUP_NAMESPACE_NAME_STR;
+        this.systemTable = true;
       } else {
         this.namespace = new byte[namespace.remaining()];
         namespace.duplicate().get(this.namespace);