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

Animesh Pandey commented on LUCENE-10197:
-----------------------------------------

Thanks for feedback on the patch. I am fine with the PR process. I actually 
thought it was only used by committers. I will create a PR in the next 
iteration even though I don't think it would include all the changes.

Since development would be going on after Nov 2021, I think it should be fine 
it goes in 2023 but I'll try to make progress as quickly as possible.

> UnifiedHighlighter should use builders for thread-safety
> --------------------------------------------------------
>
>                 Key: LUCENE-10197
>                 URL: https://issues.apache.org/jira/browse/LUCENE-10197
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/highlighter
>            Reporter: Animesh Pandey
>            Priority: Minor
>              Labels: newdev
>         Attachments: LUCENE-10197.patch
>
>
> UnifiedHighlighter is not thread-safe due to the presence of setters. We can 
> move the fields to builder so that the class becomes thread-safe.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to