[ https://issues.apache.org/jira/browse/HBASE-17250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15723787#comment-15723787 ]
huaxiang sun edited comment on HBASE-17250 at 12/5/16 11:56 PM: ---------------------------------------------------------------- Upload an initial patch. May not be a good idea to expose implementation details through getScanner() interface. But I could not find a way to better pass the flag around. There are other places such as https://github.com/apache/hbase/blob/master/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java#L6894 which similar skipCheckFamily can be applied. If folks agree that this is the right approach to take, will upload a comprehensive patch, thanks. was (Author: huaxiang): Upload an initial patch. May not be a good idea to expose implementation details through getScanner() interface. But I could find a way to better pass the flag around. There are other places such as https://github.com/apache/hbase/blob/master/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java#L6894 which similar skipCheckFamily can be applied. If folks agrees that this is the right approach to take, will upload a comprehensive patch, thanks. > For Get and scan in one case, checkFamily can be skipped in Region#getScanner > ----------------------------------------------------------------------------- > > Key: HBASE-17250 > URL: https://issues.apache.org/jira/browse/HBASE-17250 > Project: HBase > Issue Type: Improvement > Affects Versions: 2.0.0 > Reporter: huaxiang sun > Assignee: huaxiang sun > Priority: Minor > Attachments: HBASE-17250-master-001.patch > > > For get(), checkFamily is done in prepareGet(), so checkFamily can be skipped > in Region#getScanner(). For scan(), if there is no Family configured in scan, > the families are from table descriptor, so checkFamily in > Region#getScanner(). can be skipped in this case. -- This message was sent by Atlassian JIRA (v6.3.4#6332)