z-york commented on a change in pull request #2800:
URL: https://github.com/apache/hbase/pull/2800#discussion_r548209948



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
##########
@@ -246,6 +234,8 @@
   private AtomicLong compactedCellsSize = new AtomicLong();
   private AtomicLong majorCompactedCellsSize = new AtomicLong();
 
+  private HStoreContext storeContext;

Review comment:
       Yeah the idea is that HStore has way more scope than most consumers of 
HStore actually need (1. The information about the store and 2. the functions 
to mutate things - A lot of callers just care about the info, not the actual 
functions) so it made sense to separate out the info to a POJO to reduce the 
scope of what is being exposed.




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to