aboutsummaryrefslogtreecommitdiffstats
path: root/configserver/src/main
diff options
context:
space:
mode:
authorHarald Musum <musum@oath.com>2018-10-04 10:10:53 +0200
committerHarald Musum <musum@oath.com>2018-10-04 10:10:53 +0200
commitef84540e1b99cd7e1ac8bc8b18902167e1a2d1e5 (patch)
treeea0ab299590efaace651e7e949460bbe31a665ef /configserver/src/main
parent34ba19e18306f2432c61d4bcaa3a296dd38c7d7b (diff)
Remove some unnecessary logging
Diffstat (limited to 'configserver/src/main')
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/application/ZKTenantApplications.java2
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSessionRepo.java2
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/zookeeper/ZKLiveApp.java8
3 files changed, 1 insertions, 11 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ZKTenantApplications.java b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ZKTenantApplications.java
index 35c1e4fee80..466df60d688 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ZKTenantApplications.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ZKTenantApplications.java
@@ -154,8 +154,6 @@ public class ZKTenantApplications implements TenantApplications, PathChildrenCac
public void removeUnusedApplications() {
ImmutableSet<ApplicationId> activeApplications = ImmutableSet.copyOf(listApplications());
- log.log(LogLevel.DEBUG, "Removing stale applications for tenant '" + tenant +
- "', not removing these active applications: " + activeApplications);
reloadHandler.removeApplicationsExcept(activeApplications);
}
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSessionRepo.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSessionRepo.java
index fb5b5bb0be1..fe29c27abbe 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSessionRepo.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSessionRepo.java
@@ -198,7 +198,7 @@ public class RemoteSessionRepo extends SessionRepo<RemoteSession> implements Nod
break;
}
} catch (Exception e) {
- log.log(LogLevel.WARNING, session.logPre() + " error reading session id for " + applicationId);
+ log.log(LogLevel.WARNING, session.logPre() + " error reading session id for " + applicationId, e);
}
}
}
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/zookeeper/ZKLiveApp.java b/configserver/src/main/java/com/yahoo/vespa/config/server/zookeeper/ZKLiveApp.java
index 956af02e36f..383672b3639 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/zookeeper/ZKLiveApp.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/zookeeper/ZKLiveApp.java
@@ -9,7 +9,6 @@ import java.io.StringReader;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
-import java.util.logging.Level;
import java.util.logging.Logger;
/**
@@ -62,18 +61,11 @@ public class ZKLiveApp {
for (String child : children) {
if (fileNameSuffix == null || child.endsWith(fileNameSuffix)) {
result.add(new NamedReader(namePrefix + child, reader(zk.getData(fullPath, child))));
- if (log.isLoggable(Level.FINER))
- log.finer("ZKApplicationPackage: Added '" + child + "' (matched suffix " + fileNameSuffix + ")");
- } else {
- if (log.isLoggable(Level.FINER))
- log.finer("ZKApplicationPackage: Skipped '" + child + "' (did not match suffix " + fileNameSuffix + ")");
}
if (recursive)
result.addAll(getAllDataFromDirectory(path + "/" + child,
namePrefix + child + "/", fileNameSuffix, recursive));
}
- if (log.isLoggable(Level.FINE))
- log.fine("ZKApplicationPackage: Found '" + result.size() + "' files in " + fullPath);
return result;
} catch (Exception e) {
throw new RuntimeException("Could not retrieve all data from '" + fullPath + "' in zookeeper", e);