zhaih commented on a change in pull request #225:
URL: https://github.com/apache/lucene/pull/225#discussion_r724697716
##########
File path:
lucene/codecs/src/java/org/apache/lucene/codecs/memory/DirectPostingsFormat.java
##########
@@ -962,15 +964,22 @@ public ImpactsEnum impacts(int flags) throws IOException {
private int stateUpto;
public DirectIntersectTermsEnum(CompiledAutomaton compiled, BytesRef
startTerm) {
- runAutomaton = compiled.runAutomaton;
- compiledAutomaton = compiled;
+ if (compiled.nfaRunAutomaton != null) {
+ this.runAutomaton = compiled.nfaRunAutomaton;
Review comment:
I'm hesitating on doing that since there might be some method still rely
on the truth that `runAutomaton` is of type `RunAutomaton` but not only using
the method abstracted by `ByteRunnable`.
Also it might be a bit easier to spot if the nfaRunAutomaton is having
problem at this stage? Probably we want to merge it after the NFA one is more
mature?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
