SOLR-4200: Reduce INFO level logging from CachingDirectoryFactory

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1453560 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Chris M. Hostetter 2013-03-06 21:07:35 +00:00
parent d5d1603116
commit 2d0001942e
2 changed files with 7 additions and 5 deletions

View File

@ -291,6 +291,9 @@ Other Changes
* SOLR-4416: Upgrade to Tika 1.3. (Markus Jelsma via Mark Miller) * SOLR-4416: Upgrade to Tika 1.3. (Markus Jelsma via Mark Miller)
* SOLR-4200: Reduce INFO level logging from CachingDirectoryFactory
(Shawn Heisey via hossman)
================== 4.1.0 ================== ================== 4.1.0 ==================
Versions of Major Components Versions of Major Components

View File

@ -137,7 +137,7 @@ public abstract class CachingDirectoryFactory extends DirectoryFactory {
} }
assert val.refCnt == 0 : val.refCnt; assert val.refCnt == 0 : val.refCnt;
log.info("Closing directory when closing factory:" + val.path); log.info("Closing directory when closing factory: " + val.path);
closeDirectory(val); closeDirectory(val);
} catch (Throwable t) { } catch (Throwable t) {
SolrException.log(log, "Error closing directory", t); SolrException.log(log, "Error closing directory", t);
@ -158,12 +158,11 @@ public abstract class CachingDirectoryFactory extends DirectoryFactory {
throw new IllegalArgumentException("Unknown directory: " + directory throw new IllegalArgumentException("Unknown directory: " + directory
+ " " + byDirectoryCache); + " " + byDirectoryCache);
} }
log.info("Releasing directory:" + cacheValue.path); log.debug("Releasing directory: " + cacheValue.path);
cacheValue.refCnt--; cacheValue.refCnt--;
if (cacheValue.refCnt == 0 && cacheValue.doneWithDir) { if (cacheValue.refCnt == 0 && cacheValue.doneWithDir) {
log.info("Closing directory:" + cacheValue.path);
closeDirectory(cacheValue); closeDirectory(cacheValue);
byDirectoryCache.remove(directory); byDirectoryCache.remove(directory);
@ -184,7 +183,7 @@ public abstract class CachingDirectoryFactory extends DirectoryFactory {
} }
} }
try { try {
log.info("Closing directory:" + cacheValue.path); log.info("Closing directory: " + cacheValue.path);
cacheValue.directory.close(); cacheValue.directory.close();
} catch (Throwable t) { } catch (Throwable t) {
SolrException.log(log, "Error closing directory", t); SolrException.log(log, "Error closing directory", t);
@ -276,7 +275,7 @@ public abstract class CachingDirectoryFactory extends DirectoryFactory {
byDirectoryCache.put(directory, newCacheValue); byDirectoryCache.put(directory, newCacheValue);
byPathCache.put(fullPath, newCacheValue); byPathCache.put(fullPath, newCacheValue);
log.info("return new directory for " + fullPath + " forceNew:" + forceNew); log.info("return new directory for " + fullPath + " forceNew: " + forceNew);
} else { } else {
cacheValue.refCnt++; cacheValue.refCnt++;
} }