[ 
https://issues.apache.org/jira/browse/LUCENE-8159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16377952#comment-16377952
 ] 

David Smiley commented on LUCENE-8159:
--------------------------------------

bq. expose an expert constructor that takes a compiled automaton and expect 
users to compile the automaton themselves if they plan to reuse it in multiple 
queries?

+1 but I think it would be helpful to also provide access to the 
CompiledAutomaton from an existing AutomatonQuery -- e.g. 
AutomatonQuery.getCompiledAutomaton().  Not only would that be useful for 
rewriting an AutomatonQuery to change the field, but it would also be useful in 
the UnifiedHighlighter's MultiTermHighlighting line ~144 to avoid rebuilding 
the CompiledAutomaton from an existing AQ.

> Add a copy constructor in AutomatonQuery to copy directly the compiled 
> automaton
> --------------------------------------------------------------------------------
>
>                 Key: LUCENE-8159
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8159
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/search
>    Affects Versions: trunk
>            Reporter: Bruno Roustant
>            Assignee: David Smiley
>            Priority: Major
>         Attachments: 
> 0001-Add-a-copy-constructor-in-AutomatonQuery-to-copy-dir.patch, 
> LUCENE-8159.patch
>
>
> When the query is composed of multiple AutomatonQuery with the same automaton 
> and which target different fields, it is much more efficient to reuse the 
> already compiled automaton by copying it directly and just changing the 
> target field.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to