[
https://issues.apache.org/jira/browse/LUCENE-6909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15026561#comment-15026561
]
Michael McCandless commented on LUCENE-6909:
--------------------------------------------
Thanks [~sanne], I think it's safe to remove {{synchronized}} from
{{getDimConfig}}, but why change the type declaration for {{fieldTypes}}?
Can't it remain a {{Map}} (no usage requires specific methods from
{{ConcurrentHashMap}}?).
> Improve concurrency for FacetsConfig
> ------------------------------------
>
> Key: LUCENE-6909
> URL: https://issues.apache.org/jira/browse/LUCENE-6909
> Project: Lucene - Core
> Issue Type: Improvement
> Components: core/other
> Affects Versions: 5.3
> Reporter: Sanne Grinovero
> Priority: Trivial
> Attachments:
> 0001-LUCENE-6909-Allow-efficient-concurrent-usage-of-a-Fa.patch
>
>
> The design of {{org.apache.lucene.facet.FacetsConfig}} encourages reuse of a
> single instance across multiple threads, yet the current synchronization
> model is too strict as it doesn't allow for concurrent read operations.
> I'll attach a trivial patch which removes the contention point.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
