[hibernate-commits] Hibernate SVN: r14773 - search/trunk/src/java/org/hibernate/search/reader.
Author: sannegrinovero
Date: 2008-06-17 09:21:29 -0400 (Tue, 17 Jun 2008)
New Revision: 14773
Modified:
search/trunk/src/java/org/hibernate/search/reader/SharingBufferReaderProvider.java
Log:
HSEARCH-212 : new ReaderProvider final cleanups
Modified:
search/trunk/src/java/org/hibernate/search/reader/SharingBufferReaderProvider.java
===================================================================
---
search/trunk/src/java/org/hibernate/search/reader/SharingBufferReaderProvider.java 2008-06-17
13:03:13 UTC (rev 14772)
+++
search/trunk/src/java/org/hibernate/search/reader/SharingBufferReaderProvider.java 2008-06-17
13:21:29 UTC (rev 14773)
@@ -35,7 +35,7 @@
* contains all Readers (most current per DP and all unclosed old)
*/
//TODO ConcurrentHashMap's constructor could benefit from some hints as arguments.
- protected final Map<IndexReader,ReaderUsagePair> allReaders = new
ConcurrentHashMap<IndexReader,ReaderUsagePair>( 100 );
+ protected final Map<IndexReader,ReaderUsagePair> allReaders = new
ConcurrentHashMap<IndexReader,ReaderUsagePair>();
/**
* contains last updated Reader; protected by lockOnOpenLatest (in the values)
@@ -53,7 +53,7 @@
else {
throw new AssertionFailure( "Everything should be wrapped in a MultiReader"
);
}
- log.trace( "Closing MultiReader: {}", multiReader );
+ log.debug( "Closing MultiReader: {}", multiReader );
for ( IndexReader reader : readers ) {
ReaderUsagePair container = allReaders.get( reader );
container.close();//virtual
@@ -76,13 +76,14 @@
}
public IndexReader openReader(DirectoryProvider... directoryProviders) {
- boolean trace = log.isTraceEnabled();
int length = directoryProviders.length;
IndexReader[] readers = new IndexReader[length];
- if ( trace ) log.trace( "Opening IndexReader for directoryProviders: {}",
length );
- for (int index = 0; index < length; index++) {
+ log.debug( "Opening IndexReader for directoryProviders: {}", length );
+ for ( int index = 0; index < length; index++ ) {
DirectoryProvider directoryProvider = directoryProviders[index];
- if ( trace ) log.trace( "Opening IndexReader from {}",
directoryProvider.getDirectory() );
+ if ( log.isTraceEnabled() ) {
+ log.trace( "Opening IndexReader from {}", directoryProvider.getDirectory()
);
+ }
PerDirectoryLatestReader directoryLatestReader = currentReaders.get( directoryProvider
);
readers[index] = directoryLatestReader.refreshAndGet();
}