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:
[email protected]


With regards,
Apache Git Services

Reply via email to