You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2011/12/22 13:37:35 UTC

svn commit: r1222202 - /lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/SegmentCoreReaders.java

Author: simonw
Date: Thu Dec 22 12:37:35 2011
New Revision: 1222202

URL: http://svn.apache.org/viewvc?rev=1222202&view=rev
Log:
LUCENE-3653: remove synchronization on SegmentCoreReaders#getTermsReader()

Modified:
    lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/SegmentCoreReaders.java

Modified: lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/SegmentCoreReaders.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/SegmentCoreReaders.java?rev=1222202&r1=1222201&r2=1222202&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/SegmentCoreReaders.java (original)
+++ lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/SegmentCoreReaders.java Thu Dec 22 12:37:35 2011
@@ -53,7 +53,7 @@ final class SegmentCoreReaders {
 
   private final SegmentReader owner;
 
-  TermInfosReader tis;
+  volatile TermInfosReader tis;
   FieldsReader fieldsReaderOrig;
   TermVectorsReader termVectorsReaderOrig;
   CompoundFileReader cfsReader;
@@ -127,7 +127,8 @@ final class SegmentCoreReaders {
     return cfsReader;
   }
 
-  synchronized TermInfosReader getTermsReader() {
+  TermInfosReader getTermsReader() {
+    final TermInfosReader tis = this.tis;
     if (tis != null) {
       return tis;
     } else {
@@ -135,7 +136,7 @@ final class SegmentCoreReaders {
     }
   }      
 
-  synchronized boolean termsIndexIsLoaded() {
+  boolean termsIndexIsLoaded() {
     return tis != null;
   }