Github user joshelser commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/323#discussion_r208711828
--- 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);
+ if(memstoreScanners != null) {
--- End diff --
What is the case that this would ever be null?
---
