[
https://issues.apache.org/jira/browse/LUCENE-6909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sanne Grinovero updated LUCENE-6909:
------------------------------------
Attachment: 0001-LUCENE-6909-Allow-efficient-concurrent-usage-of-a-Fa.patch
Trivial patch.
The synchronization isn't needed on `getDimConfig` because it's reading from a
ConcurrentMap.
Synchronization is still needed on setters, but that's not a performance
concern as the usage pattern is supposedly to configure the fields once and
then reuse the instance mostly reading.
> 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]
