rmuir commented on issue #13715:
URL: https://github.com/apache/lucene/issues/13715#issuecomment-2329919715

   @ChrisHegarty I think it might be an oversight IntervalQuery is getting an 
NFA, was that really intended?
   
   The following other methods on intervals are powered by automatons and get a 
DFA:
   * Intervals.prefix()
   * Intervals.wildcard()
   * Intervals.range()
   * Intervals.fuzzyTerm()
   
   This happens because they reuse the parsing from the associated Query 
classes, but this Intervals.regexp() neglects to do that and just creates its 
own RegExp and converts it to an NFA. Given that the default of RegexpQuery is 
to determinize, I think we should determinize here to avoid surprises such as 
this?


-- 
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: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to