diff options
author | Jon Bratseth <bratseth@gmail.com> | 2022-10-27 00:21:57 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2022-10-27 00:21:57 +0200 |
commit | 4ad1d0ca013b925858597c559ab4efb02c30dd4a (patch) | |
tree | dcb83bab5359f7f508e9ecb9a919e831046ccdff /configserver | |
parent | 0d4c548f548ee7fd904eb5cabc4c0a68cbad85ec (diff) |
Use IllegalArgumentException when appropriate
Diffstat (limited to 'configserver')
2 files changed, 4 insertions, 3 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java b/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java index 0b409d38196..5ceeb83d2d3 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java @@ -157,8 +157,8 @@ public class Application implements ModelResult { // This can happen in cases where services ask for config that no longer exist before they have been able // to reconfigure themselves log.log(Level.INFO, TenantRepository.logPre(getId()) + - ": Error resolving instance for builder '" + cacheBuilderClassNameForErrorReport + - "', returning empty config: " + Exceptions.toMessageString(e)); + ": Error resolving instance for builder '" + cacheBuilderClassNameForErrorReport + + "', returning empty config: " + Exceptions.toMessageString(e)); payload = ConfigPayload.fromBuilder(new ConfigPayloadBuilder()); } if (def.getCNode() != null) diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ConfigInstanceBuilder.java b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ConfigInstanceBuilder.java index 1167f32caf5..d99d9a7e017 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ConfigInstanceBuilder.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ConfigInstanceBuilder.java @@ -109,7 +109,8 @@ class ConfigInstanceBuilder { private static Class<? extends ConfigInstance> getConfigClass(Class<? extends ConfigInstance.Builder> builderClass) { Class<?> configClass = builderClass.getEnclosingClass(); if (configClass == null || ! ConfigInstance.class.isAssignableFrom(configClass)) { - throw new ConfigurationRuntimeException("Builder class " + builderClass + " has enclosing class " + configClass + ", which is not a ConfigInstance"); + throw new ConfigurationRuntimeException("Builder class " + builderClass + " has enclosing class " + + configClass + ", which is not a ConfigInstance"); } return (Class<? extends ConfigInstance>) configClass; } |