Github user mikemccand commented on a diff in the pull request:
https://github.com/apache/lucene-solr/pull/286#discussion_r156538059
--- Diff:
lucene/core/src/java/org/apache/lucene/codecs/blocktree/IntersectTermsEnum.java
---
@@ -106,37 +106,37 @@ public IntersectTermsEnum(FieldReader fr, Automaton
automaton, RunAutomaton runA
if (fr.index == null) {
fstReader = null;
--- End diff --
Sorry for the slow respons ehere @imgpulak and @jpountz but Adrien is
right: `fr.index` can never be null anymore. So I think we should change the
code to `assert` it's never null and only do the `else` clause of the current
`if` statement?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
