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;
}