Github user mnpoonia commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/323#discussion_r209157109
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/hadoop/hbase/regionserver/IndexHalfStoreFileReaderGenerator.java
 ---
    @@ -395,7 +395,10 @@ public KeyValueScanner preStoreScannerOpen(final 
ObserverContext<RegionCoprocess
                             .getReader().getHFileReader().hasMVCCInfo(), 
readPt));
                 }
             }
    -        keyValueScanners.addAll(((HStore) 
store).memstore.getScanners(readPt));
    +        List<KeyValueScanner> memstoreScanners = ((HStore) 
store).memstore.getScanners(readPt);
    --- End diff --
    
    @joshelser Looks like we need HStore to access memstore specifically. We 
can't access the memstore and get its scanners from the store object. 


---

Reply via email to