diff options
author | Jon Bratseth <bratseth@yahoo-inc.com> | 2016-09-05 12:00:04 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@yahoo-inc.com> | 2016-09-05 12:00:04 +0200 |
commit | dc5a90cc5a085edd5ff7a413afea8c0c9b4fdccb (patch) | |
tree | c98fb9163579a471ef5bb35a0b7a81796bda6178 /config/src/main | |
parent | cbb0b56aab19a7e44b5b16055618bba9245795e4 (diff) |
Don't support missing namespace
Diffstat (limited to 'config/src/main')
-rw-r--r-- | config/src/main/java/com/yahoo/vespa/config/ConfigDefinitionKey.java | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/config/src/main/java/com/yahoo/vespa/config/ConfigDefinitionKey.java b/config/src/main/java/com/yahoo/vespa/config/ConfigDefinitionKey.java index 29c39ef0ab0..c7396394f62 100644 --- a/config/src/main/java/com/yahoo/vespa/config/ConfigDefinitionKey.java +++ b/config/src/main/java/com/yahoo/vespa/config/ConfigDefinitionKey.java @@ -1,6 +1,8 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.config; +import java.util.Objects; + /** * Represents one config definition key (name, namespace) * @@ -13,13 +15,22 @@ public class ConfigDefinitionKey { /** * Creates a config definition key. - * @param name config definition name + * + * @param name config definition name * @param namespace config definition namespace */ public ConfigDefinitionKey(String name, String namespace) { + require(name, "A config name cannot be null or empty"); + require(namespace, "A config name cannot be null or empty"); this.name = name; this.namespace = namespace; } + + private static void require(String object, String message) { + Objects.requireNonNull(object, message); + if (object.isEmpty()) + throw new IllegalArgumentException(message); + } public ConfigDefinitionKey(ConfigKey<?> key) { this(key.getName(), key.getNamespace()); |