diff options
author | Harald Musum <musum@oath.com> | 2018-10-04 10:10:53 +0200 |
---|---|---|
committer | Harald Musum <musum@oath.com> | 2018-10-04 10:10:53 +0200 |
commit | ef84540e1b99cd7e1ac8bc8b18902167e1a2d1e5 (patch) | |
tree | ea0ab299590efaace651e7e949460bbe31a665ef /configserver/src/main | |
parent | 34ba19e18306f2432c61d4bcaa3a296dd38c7d7b (diff) |
Remove some unnecessary logging
Diffstat (limited to 'configserver/src/main')
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); |