You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2015/11/25 17:35:56 UTC
svn commit: r1716477 - in /lucene/dev/branches/branch_5x: ./ lucene/
lucene/CHANGES.txt lucene/facet/
lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java
Author: mikemccand
Date: Wed Nov 25 16:35:56 2015
New Revision: 1716477
URL: http://svn.apache.org/viewvc?rev=1716477&view=rev
Log:
LUCENE-6909: remove unnecessary synchronized keyword
Modified:
lucene/dev/branches/branch_5x/ (props changed)
lucene/dev/branches/branch_5x/lucene/ (props changed)
lucene/dev/branches/branch_5x/lucene/CHANGES.txt (contents, props changed)
lucene/dev/branches/branch_5x/lucene/facet/ (props changed)
lucene/dev/branches/branch_5x/lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java
Modified: lucene/dev/branches/branch_5x/lucene/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/CHANGES.txt?rev=1716477&r1=1716476&r2=1716477&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/CHANGES.txt (original)
+++ lucene/dev/branches/branch_5x/lucene/CHANGES.txt Wed Nov 25 16:35:56 2015
@@ -136,6 +136,10 @@ Optimizations
be fully read from disk if the supplied StoredFieldVisitor doesn't want it.
So put your largest text field value last to benefit. (David Smiley)
+* LUCENE-6909: Remove unnecessary synchronized from
+ FacetsConfig.getDimConfig for better concurrency (Sanne Grinovero
+ via Mike McCandless)
+
Bug Fixes
* LUCENE-6817: ComplexPhraseQueryParser.ComplexPhraseQuery does not display
Modified: lucene/dev/branches/branch_5x/lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java?rev=1716477&r1=1716476&r2=1716477&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java (original)
+++ lucene/dev/branches/branch_5x/lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java Wed Nov 25 16:35:56 2015
@@ -112,7 +112,7 @@ public class FacetsConfig {
}
/** Get the current configuration for a dimension. */
- public synchronized DimConfig getDimConfig(String dimName) {
+ public DimConfig getDimConfig(String dimName) {
DimConfig ft = fieldTypes.get(dimName);
if (ft == null) {
ft = getDefaultDimConfig();