[GitHub] [hbase] bharathv commented on a change in pull request #995: HBASE-23055 Alter hbase:meta

[GitBox <gi...@apache.org>]

bharathv commented on a change in pull request #995: HBASE-23055 Alter hbase:meta
URL: https://github.com/apache/hbase/pull/995#discussion_r363613879
 
 

 ##########
 File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncRegistry.java
 ##########
 @@ -29,12 +30,26 @@
  */
 @InterfaceAudience.Private
 interface AsyncRegistry extends Closeable {
+  /**
+   * A completed CompletableFuture to host default hbase:meta table state (ENABLED).
+   */
+  TableState ENABLED_META_TABLE_STATE =
+    new TableState(TableName.META_TABLE_NAME, TableState.State.ENABLED);
+  CompletableFuture<TableState> COMPLETED_GET_META_TABLE_STATE =
+    CompletableFuture.completedFuture(ENABLED_META_TABLE_STATE);
 
   /**
    * Get the location of meta region.
    */
   CompletableFuture<RegionLocations> getMetaRegionLocation();
 
+  /**
+   * The hbase:meta table state.
+   */
+  default CompletableFuture<TableState> getMetaTableState() {
 
 Review comment:
   More of a question: whats the need for a default implementation? This is almost always overriden by an implementation?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services