Merge pull request #11 from safarijv/1.6.2

v1.6.2 Always use getSlowAtomicReader() instead of top reader context
oreillymedia · Jul 10, 2019 · 2c741a6 · 2c741a6
2 parents c4533d3 + b17d621
commit 2c741a6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/pom.xml b/pom.xml
@@ -2,7 +2,7 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>ifpress</groupId>
   <artifactId>ifpress-solr-plugin</artifactId>
-  <version>1.6.1</version>
+  <version>1.6.2</version>
   <name>ifpress solr plugin</name>
   <description>Contains plugins to be installed in the solr server</description>
   <dependencies>

diff --git a/src/main/java/com/ifactory/press/db/solr/processor/UpdateDocValuesProcessor.java b/src/main/java/com/ifactory/press/db/solr/processor/UpdateDocValuesProcessor.java
@@ -125,9 +125,7 @@ private void retrieveDocValuesHelper(String[] valueFields, SolrInputDocument doc
     Term idTerm = new Term(idField, id);
     TermQuery query = new TermQuery (idTerm);
     TopDocs docs = searcher.search(query, 1);
-    // getSlowAtomicReader is slower, so try to get LeafReader from LeafReaderContext first
-    List<LeafReaderContext> leaves = searcher.getTopReaderContext().leaves();
-    LeafReader leafReader = leaves.isEmpty() ? searcher.getSlowAtomicReader() : leaves.get(0).reader();
+    LeafReader leafReader = searcher.getSlowAtomicReader();
     if (docs.totalHits == 1) {
       // get the value
       // LOG.debug(String.format("found %s", id));