SOLR-12464: Reduce Overseer.close() logging (for non-Overseer leaders)

This commit is contained in:
Christine Poerschke 2018-07-17 17:09:07 +01:00
parent 8093c450c1
commit f7aed28760
2 changed files with 11 additions and 3 deletions

View File

@ -183,6 +183,8 @@ Other Changes
* SOLR-12551: Upgrade to Tika 1.18 (Tim Allison via Erick Erickson) * SOLR-12551: Upgrade to Tika 1.18 (Tim Allison via Erick Erickson)
* SOLR-12464: Reduce Overseer.close() logging (for non-Overseer leaders) (Christine Poerschke)
================== 7.4.0 ================== ================== 7.4.0 ==================
Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release. Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.

View File

@ -544,8 +544,10 @@ public class Overseer implements SolrCloseable {
updaterThread.start(); updaterThread.start();
ccThread.start(); ccThread.start();
triggerThread.start(); triggerThread.start();
if (this.id != null) {
assert ObjectReleaseTracker.track(this); assert ObjectReleaseTracker.track(this);
} }
}
public Stats getStats() { public Stats getStats() {
return stats; return stats;
@ -584,12 +586,16 @@ public class Overseer implements SolrCloseable {
public synchronized void close() { public synchronized void close() {
if (closed) return; if (closed) return;
if (this.id != null) {
log.info("Overseer (id=" + id + ") closing"); log.info("Overseer (id=" + id + ") closing");
}
doClose(); doClose();
this.closed = true; this.closed = true;
if (this.id != null) {
assert ObjectReleaseTracker.release(this); assert ObjectReleaseTracker.release(this);
} }
}
@Override @Override
public boolean isClosed() { public boolean isClosed() {