saintstack commented on a change in pull request #957: HBASE-23604: Clarify
AsyncRegistry usage in the code.
URL: https://github.com/apache/hbase/pull/957#discussion_r360654175
##########
File path:
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionMetaRegistry.java
##########
@@ -24,16 +24,17 @@
import org.apache.yetus.audience.InterfaceAudience;
/**
- * Implementations hold cluster information such as this cluster's id,
location of hbase:meta, etc..
+ * Registry for meta information needed for connection setup to a HBase
cluster. Implementations
+ * hold cluster information such as this cluster's id, location of hbase:meta,
etc..
* Internal use only.
*/
@InterfaceAudience.Private
-interface AsyncRegistry extends Closeable {
+interface AsyncConnectionMetaRegistry extends Closeable {
/**
- * Get the location of meta region.
+ * Get the current locations of meta region(s).
*/
- CompletableFuture<RegionLocations> getMetaRegionLocation();
+ CompletableFuture<RegionLocations> getMetaRegionLocations();
Review comment:
This is good.
----------------------------------------------------------------
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
