Jonathan Hsieh created HBASE-15061:
--------------------------------------

             Summary: Refactor StoreFileScanner creation to builder pattern
                 Key: HBASE-15061
                 URL: https://issues.apache.org/jira/browse/HBASE-15061
             Project: HBase
          Issue Type: Improvement
    Affects Versions: 2.0.0
            Reporter: Jonathan Hsieh
            Assignee: Jonathan Hsieh
             Fix For: 2.0.0


There are several falvors of  calls that creates a list of StoreFileScanners, 
and new feature have been added to this recently.  This patch converts the 
somewhat difficult to read (need to go to javadoc) call:

{code}
// which args are the most relevant to this?
-      List<StoreFileScanner> sfScanners = 
StoreFileScanner.getScannersForStoreFiles(sfs,
-        cacheMobBlocks, true, false, false, readPt);
{code}

into one that is more literate:

{code}
// ah, very clearly we are using defaults except for the caching settings
+      List<StoreFileScanner> sfScanners = new 
StoreFileScanner.ListBuilder(sfs, readPt)
+          .withCacheBlocks(cacheMobBlocks).build();
{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to