You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by GitBox <gi...@apache.org> on 2021/06/25 16:15:37 UTC

[GitHub] [hbase] anoopsjohn commented on a change in pull request #3426: HBASE-26032 Make HRegion.getStores() an O(1) operation

anoopsjohn commented on a change in pull request #3426:
URL: https://github.com/apache/hbase/pull/3426#discussion_r658884218



##########
File path: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/Region.java
##########
@@ -92,7 +92,7 @@
    * <p>Use with caution.  Exposed for use of fixup utilities.
    * @return a list of the Stores managed by this region
    */
-  List<? extends Store> getStores();
+  Collection<? extends Store> getStores();

Review comment:
       Region is exposed to CPs.  
   Also that is the reason why direct internal data structure is not exposed I believe.




-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org