diff options
49 files changed, 19 insertions, 512 deletions
diff --git a/config-model/src/main/java/com/yahoo/config/model/producer/AbstractConfigProducer.java b/config-model/src/main/java/com/yahoo/config/model/producer/AbstractConfigProducer.java index 23aa7e80d74..b86230c97e6 100644 --- a/config-model/src/main/java/com/yahoo/config/model/producer/AbstractConfigProducer.java +++ b/config-model/src/main/java/com/yahoo/config/model/producer/AbstractConfigProducer.java @@ -6,7 +6,6 @@ import com.yahoo.config.ConfigInstance; import com.yahoo.config.model.ApplicationConfigProducerRoot; import com.yahoo.config.model.deploy.DeployState; import com.yahoo.config.subscription.ConfigInstanceUtil; -import java.util.logging.Level; import com.yahoo.vespa.config.ConfigDefinitionKey; import com.yahoo.vespa.config.ConfigPayload; import com.yahoo.vespa.config.ConfigPayloadBuilder; @@ -19,7 +18,6 @@ import com.yahoo.vespa.model.SimpleConfigProducer; import com.yahoo.vespa.model.admin.Admin; import com.yahoo.vespa.model.admin.monitoring.Monitoring; import com.yahoo.vespa.model.utils.FreezableMap; - import java.io.PrintStream; import java.io.Serializable; import java.util.ArrayList; @@ -27,6 +25,7 @@ import java.util.Collections; import java.util.LinkedHashMap; import java.util.List; import java.util.Map; +import java.util.logging.Level; import java.util.logging.Logger; /** @@ -233,7 +232,6 @@ public abstract class AbstractConfigProducer<CHILD extends AbstractConfigProduce return didApply; } - @SuppressWarnings("removal") // TODO Vespa 8: remove private void applyUserConfig(ConfigInstance.Builder builder, ConfigPayloadBuilder payloadBuilder) { ConfigInstance.Builder override; if (builder instanceof GenericConfig.GenericConfigBuilder) { diff --git a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java index 8f844fdd3ee..02c5aeb33c7 100644 --- a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java +++ b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/RpcConfigSourceClient.java @@ -47,7 +47,6 @@ class RpcConfigSourceClient implements ConfigSourceClient, Runnable { private final Supervisor supervisor = new Supervisor(new Transport("config-source-client")); private final ResponseHandler responseHandler; - @SuppressWarnings("removal") // TODO Vespa 8: remove private final ConfigSourceSet configSourceSet; private final Object subscribersLock = new Object(); private final Map<ConfigCacheKey, Subscriber> subscribers = new ConcurrentHashMap<>(); @@ -62,7 +61,6 @@ class RpcConfigSourceClient implements ConfigSourceClient, Runnable { Executors.newScheduledThreadPool(1, new DaemonThreadFactory("delayed responses")); private final ScheduledFuture<?> delayedResponsesFuture; - @SuppressWarnings("removal") // TODO Vespa 8: remove RpcConfigSourceClient(ResponseHandler responseHandler, ConfigSourceSet configSourceSet) { this.responseHandler = responseHandler; this.configSourceSet = configSourceSet; diff --git a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionAndUrlDownload.java b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionAndUrlDownload.java index 8e471aa1f5d..68570722117 100644 --- a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionAndUrlDownload.java +++ b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionAndUrlDownload.java @@ -26,7 +26,6 @@ public class FileDistributionAndUrlDownload { private final ScheduledExecutorService cleanupExecutor = new ScheduledThreadPoolExecutor(1, new DaemonThreadFactory("file references and downloads cleanup")); - @SuppressWarnings("removal") // TODO Vespa 8: remove public FileDistributionAndUrlDownload(Supervisor supervisor, ConfigSourceSet source) { fileDistributionRpcServer = new FileDistributionRpcServer(supervisor, createDownloader(supervisor, source)); urlDownloadRpcServer = new UrlDownloadRpcServer(supervisor); @@ -45,7 +44,6 @@ public class FileDistributionAndUrlDownload { } } - @SuppressWarnings("removal") // TODO Vespa 8: remove private FileDownloader createDownloader(Supervisor supervisor, ConfigSourceSet source) { return new FileDownloader(new FileDistributionConnectionPool(source, supervisor), supervisor, diff --git a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ConfigProxyRpcServerTest.java b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ConfigProxyRpcServerTest.java index 2105bd72975..f743c0ed231 100644 --- a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ConfigProxyRpcServerTest.java +++ b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ConfigProxyRpcServerTest.java @@ -256,7 +256,6 @@ public class ConfigProxyRpcServerTest { assertEquals("success", req.returnValues().get(0).asString()); } - @SuppressWarnings("removal") // TODO Vespa 8: remove private static ProxyServer createTestServer(ConfigSourceSet source) { return new ProxyServer(null, source, new RpcConfigSourceClient(new ResponseHandler(), source)); } @@ -265,7 +264,6 @@ public class ConfigProxyRpcServerTest { private static final Spec SPEC = new Spec(0); - @SuppressWarnings("removal") // TODO Vespa 8: remove private final ProxyServer proxyServer = createTestServer(new ConfigSourceSet(configSourceAddress)); private final Supervisor supervisor = new Supervisor(new Transport()); private final ConfigProxyRpcServer rpcServer = new ConfigProxyRpcServer(proxyServer, supervisor, SPEC); diff --git a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/MockConfigSource.java b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/MockConfigSource.java index 2b3624fe2b8..0377e20330a 100644 --- a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/MockConfigSource.java +++ b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/MockConfigSource.java @@ -15,7 +15,6 @@ import java.util.Set; * * @author hmusum */ -@SuppressWarnings("removal") // TODO Vespa 8: remove class MockConfigSource extends ConfigSourceSet { private final HashMap<ConfigKey<?>, RawConfig> backing = new HashMap<>(); diff --git a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ProxyServerTest.java b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ProxyServerTest.java index 1bcd1562a85..0e25e62b925 100644 --- a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ProxyServerTest.java +++ b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/ProxyServerTest.java @@ -216,7 +216,6 @@ public class ProxyServerTest { assertEquals(ProxyServer.DEFAULT_PROXY_CONFIG_SOURCES, properties.configSources[0]); } - @SuppressWarnings("removal") // TODO Vespa 8: remove private static ProxyServer createTestServer(ConfigSourceSet source, ConfigSourceClient configSourceClient) { return new ProxyServer(null, source, configSourceClient); } diff --git a/config/abi-spec.json b/config/abi-spec.json index e94749cfba8..9e26dfeeb6e 100644 --- a/config/abi-spec.json +++ b/config/abi-spec.json @@ -1,312 +1 @@ -{ - "com.yahoo.config.subscription.CfgConfigPayloadBuilder": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()", - "public com.yahoo.vespa.config.ConfigPayload deserialize(java.util.List)", - "public com.yahoo.vespa.config.ConfigPayloadBuilder deserializeToBuilder(java.util.List)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigDebug": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()", - "public static void logDebug(java.util.logging.Logger, long, com.yahoo.vespa.config.ConfigKey, java.lang.String)", - "public static void logDebug(java.util.logging.Logger, com.yahoo.config.ConfigInstance$Builder, java.lang.String, java.lang.String)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigGetter": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(java.lang.Class)", - "public void <init>(com.yahoo.config.subscription.ConfigSource, java.lang.Class)", - "public synchronized com.yahoo.config.ConfigInstance getConfig(java.lang.String)", - "public static com.yahoo.config.ConfigInstance getConfig(java.lang.Class, java.lang.String)", - "public static com.yahoo.config.ConfigInstance getConfig(java.lang.Class, java.lang.String, com.yahoo.config.subscription.ConfigSource)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigHandle": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "protected void <init>(com.yahoo.config.subscription.impl.ConfigSubscription)", - "public boolean isChanged()", - "public com.yahoo.config.ConfigInstance getConfig()", - "public java.lang.String toString()" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigInstanceSerializer": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.Serializer" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(com.yahoo.slime.Slime)", - "public void <init>(com.yahoo.slime.Slime, com.yahoo.slime.Cursor)", - "public com.yahoo.config.Serializer createInner(java.lang.String)", - "public com.yahoo.config.Serializer createArray(java.lang.String)", - "public com.yahoo.config.Serializer createInner()", - "public com.yahoo.config.Serializer createMap(java.lang.String)", - "public void serialize(java.lang.String, boolean)", - "public void serialize(java.lang.String, double)", - "public void serialize(java.lang.String, int)", - "public void serialize(java.lang.String, long)", - "public void serialize(java.lang.String, java.lang.String)", - "public void serialize(boolean)", - "public void serialize(double)", - "public void serialize(long)", - "public void serialize(int)", - "public void serialize(java.lang.String)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigInstanceUtil": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()", - "public static void setValues(com.yahoo.config.ConfigBuilder, com.yahoo.config.ConfigBuilder)", - "public static com.yahoo.config.ConfigInstance getNewInstance(java.lang.Class, java.lang.String, com.yahoo.vespa.config.ConfigPayload)", - "public static java.lang.Object getField(com.yahoo.config.ConfigBuilder, java.lang.String)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigInterruptedException": { - "superClass": "java.lang.RuntimeException", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(java.lang.Throwable)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigSet": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.subscription.ConfigSource" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()", - "public void addBuilder(java.lang.String, com.yahoo.config.ConfigInstance$Builder)", - "public com.yahoo.config.ConfigInstance$Builder get(com.yahoo.vespa.config.ConfigKey)", - "public boolean contains(com.yahoo.vespa.config.ConfigKey)", - "public java.lang.String toString()" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigSource": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public", - "interface", - "abstract" - ], - "methods": [], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigSourceSet": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.subscription.ConfigSource" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()", - "public void <init>(java.util.List)", - "public void <init>(java.lang.String[])", - "public void <init>(java.lang.String)", - "public java.util.Set getSources()", - "public boolean equals(java.lang.Object)", - "public int hashCode()", - "public java.lang.String toString()", - "public static com.yahoo.config.subscription.ConfigSourceSet createDefault()" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigSubscriber$SingleSubscriber": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public", - "interface", - "abstract" - ], - "methods": [ - "public abstract void configure(com.yahoo.config.ConfigInstance)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.ConfigSubscriber$State": { - "superClass": "java.lang.Enum", - "interfaces": [], - "attributes": [ - "public", - "final", - "enum" - ], - "methods": [ - "public static com.yahoo.config.subscription.ConfigSubscriber$State[] values()", - "public static com.yahoo.config.subscription.ConfigSubscriber$State valueOf(java.lang.String)" - ], - "fields": [ - "public static final enum com.yahoo.config.subscription.ConfigSubscriber$State OPEN", - "public static final enum com.yahoo.config.subscription.ConfigSubscriber$State FROZEN", - "public static final enum com.yahoo.config.subscription.ConfigSubscriber$State CLOSED" - ] - }, - "com.yahoo.config.subscription.ConfigSubscriber": { - "superClass": "java.lang.Object", - "interfaces": [ - "java.lang.AutoCloseable" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()", - "public void <init>(com.yahoo.config.subscription.ConfigSource)", - "public com.yahoo.config.subscription.ConfigHandle subscribe(java.lang.Class, java.lang.String)", - "public com.yahoo.config.subscription.ConfigHandle subscribe(java.lang.Class, java.lang.String, long)", - "protected void checkStateBeforeSubscribe()", - "protected void subscribeAndHandleErrors(com.yahoo.config.subscription.impl.ConfigSubscription, com.yahoo.vespa.config.ConfigKey, com.yahoo.config.subscription.ConfigHandle, com.yahoo.vespa.config.TimingValues)", - "public boolean nextConfig(boolean)", - "public boolean nextConfig()", - "public boolean nextConfig(long, boolean)", - "public boolean nextConfig(long)", - "public boolean nextGeneration(boolean)", - "public boolean nextGeneration()", - "public boolean nextGeneration(long, boolean)", - "public boolean nextGeneration(long)", - "protected void throwIfExceptionSet(com.yahoo.config.subscription.impl.ConfigSubscription)", - "public void close()", - "public java.lang.String toString()", - "public java.lang.Thread startConfigThread(java.lang.Runnable)", - "protected com.yahoo.config.subscription.ConfigSubscriber$State state()", - "public void reload(long)", - "public com.yahoo.config.subscription.ConfigSource getSource()", - "public boolean isClosed()", - "public com.yahoo.config.subscription.ConfigHandle subscribe(com.yahoo.config.subscription.ConfigSubscriber$SingleSubscriber, java.lang.Class, java.lang.String)", - "public long getGeneration()", - "protected void finalize()" - ], - "fields": [ - "protected final java.util.List subscriptionHandles" - ] - }, - "com.yahoo.config.subscription.ConfigURI": { - "superClass": "java.lang.Object", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public java.lang.String getConfigId()", - "public com.yahoo.config.subscription.ConfigSource getSource()", - "public static com.yahoo.config.subscription.ConfigURI createFromId(java.lang.String)", - "public static com.yahoo.config.subscription.ConfigURI createFromIdAndSource(java.lang.String, com.yahoo.config.subscription.ConfigSource)" - ], - "fields": [] - }, - "com.yahoo.config.subscription.DirSource": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.subscription.ConfigSource" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(java.io.File)", - "public java.io.File getDir()" - ], - "fields": [] - }, - "com.yahoo.config.subscription.FileSource": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.subscription.ConfigSource" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(java.io.File)", - "public java.io.File getFile()" - ], - "fields": [] - }, - "com.yahoo.config.subscription.JarSource": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.subscription.ConfigSource" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(java.util.jar.JarFile, java.lang.String)", - "public java.util.jar.JarFile getJarFile()", - "public java.lang.String getPath()" - ], - "fields": [] - }, - "com.yahoo.config.subscription.RawSource": { - "superClass": "java.lang.Object", - "interfaces": [ - "com.yahoo.config.subscription.ConfigSource" - ], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>(java.lang.String)" - ], - "fields": [ - "public final java.lang.String payload" - ] - }, - "com.yahoo.config.subscription.SubscriberClosedException": { - "superClass": "java.lang.RuntimeException", - "interfaces": [], - "attributes": [ - "public" - ], - "methods": [ - "public void <init>()" - ], - "fields": [] - } -}
\ No newline at end of file +{}
\ No newline at end of file diff --git a/config/pom.xml b/config/pom.xml index 546e65c048c..4eceef8c748 100755 --- a/config/pom.xml +++ b/config/pom.xml @@ -186,14 +186,6 @@ <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-compiler-plugin</artifactId> - <!-- TODO Vespa 8: remove configuration. - Included to allow 'removal' warnings for classes in its own module --> - <configuration> - <compilerArgs> - <arg>-Xlint:all,-serial,-try,-processing,-removal</arg> - <arg>-Werror</arg> - </compilerArgs> - </configuration> </plugin> <plugin> <groupId>org.apache.maven.plugins</groupId> diff --git a/config/src/main/java/com/yahoo/config/subscription/CfgConfigPayloadBuilder.java b/config/src/main/java/com/yahoo/config/subscription/CfgConfigPayloadBuilder.java index 7d3fdad43c7..1a110fe8e6c 100644 --- a/config/src/main/java/com/yahoo/config/subscription/CfgConfigPayloadBuilder.java +++ b/config/src/main/java/com/yahoo/config/subscription/CfgConfigPayloadBuilder.java @@ -17,9 +17,7 @@ import static java.util.logging.Level.FINEST; * Deserializes config payload (cfg format) to a ConfigPayload. * * @author hmusum - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class CfgConfigPayloadBuilder { private static final java.util.logging.Logger log = java.util.logging.Logger.getLogger(CfgConfigPayloadBuilder.class.getName()); diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigDebug.java b/config/src/main/java/com/yahoo/config/subscription/ConfigDebug.java index 2fcbdce7030..85aa7a05e17 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigDebug.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigDebug.java @@ -9,7 +9,6 @@ import java.util.logging.Logger; import static java.util.logging.Level.INFO; // Debug class that provides useful helper routines -@Deprecated(forRemoval = true, since = "7") public class ConfigDebug { public static void logDebug(Logger logger, long timestamp, ConfigKey<?> key, String logmessage) { if (key.getConfigId().matches(".*container.?\\d+.*") || key.getConfigId().matches(".*doc.api.*")) { diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigGetter.java b/config/src/main/java/com/yahoo/config/subscription/ConfigGetter.java index 305ecbd6483..aabfd211fac 100755 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigGetter.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigGetter.java @@ -17,7 +17,6 @@ import com.yahoo.config.ConfigInstance; public class ConfigGetter<T extends ConfigInstance> { private final Class<T> clazz; - private final ConfigSource source; /** * Creates a ConfigGetter for class <code>clazz</code> @@ -25,22 +24,7 @@ public class ConfigGetter<T extends ConfigInstance> { * @param clazz a config class */ public ConfigGetter(Class<T> clazz) { - this(null, clazz); - } - - /** - * Creates a ConfigGetter for class <code>clazz</code> with the specified - * {@link ConfigSource}. - * - * @param source a {@link ConfigSource} - * @param clazz a config class - * - * @deprecated Config should always be injected via the component class constructor. For unit tests, use config builders. - */ - @Deprecated(forRemoval = true, since = "7") - public ConfigGetter(ConfigSource source, Class<T> clazz) { this.clazz = clazz; - this.source = source; } /** @@ -50,8 +34,7 @@ public class ConfigGetter<T extends ConfigInstance> { * @return an instance of a config class */ public synchronized T getConfig(String configId) { - try (ConfigSubscriber subscriber = - source == null ? new ConfigSubscriber() : new ConfigSubscriber(source)) { + try (ConfigSubscriber subscriber = new ConfigSubscriber()) { ConfigHandle<T> handle = subscriber.subscribe(clazz, configId); subscriber.nextConfig(true); return handle.getConfig(); @@ -70,19 +53,4 @@ public class ConfigGetter<T extends ConfigInstance> { return getter.getConfig(configId); } - /** - * Creates a ConfigGetter instance and returns an instance of the config class <code>c</code>. - * - * @param c a config class - * @param configId a config id to use when getting the config - * @param source a {@link ConfigSource} - * @return an instance of a config class - * - * @deprecated Config should always be injected via the component class constructor. For unit tests, use config builders. - */ - @Deprecated(forRemoval = true, since = "7") - public static <T extends ConfigInstance> T getConfig(Class<T> c, String configId, ConfigSource source) { - ConfigGetter<T> getter = new ConfigGetter<>(source, c); - return getter.getConfig(configId); - } } diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigHandle.java b/config/src/main/java/com/yahoo/config/subscription/ConfigHandle.java index 6b832205211..4ae6122becd 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigHandle.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigHandle.java @@ -6,13 +6,11 @@ import com.yahoo.config.subscription.impl.ConfigSubscription; /** * A config handle represents one config in the context of one active subscription on a {@link ConfigSubscriber}. - * It will contain meta data of the subscription of that particular config, as well as access to the {@link com.yahoo.config.ConfigInstance} itself. + * It will contain metadata of the subscription of that particular config, as well as access to the {@link com.yahoo.config.ConfigInstance} itself. * * @param <T> the type of the config * @author vegardh - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigHandle<T extends ConfigInstance> { private final ConfigSubscription<T> sub; diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceSerializer.java b/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceSerializer.java index d8062a9f95b..e035991abf4 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceSerializer.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceSerializer.java @@ -9,9 +9,7 @@ import com.yahoo.slime.Slime; * Implements a config instance serializer, serializing a config instance to a slime object. * * @author Ulf Lilleengen - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigInstanceSerializer implements Serializer { private final Slime slime; private final Cursor root; diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceUtil.java b/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceUtil.java index 5831753eacb..61b421b30e0 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceUtil.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigInstanceUtil.java @@ -1,22 +1,19 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.config.subscription; -import java.lang.reflect.Field; -import java.lang.reflect.Method; -import java.lang.reflect.Constructor; -import java.lang.reflect.InvocationTargetException; - import com.yahoo.config.ConfigBuilder; import com.yahoo.config.ConfigInstance; import com.yahoo.config.ConfigurationRuntimeException; -import com.yahoo.yolean.Exceptions; -import com.yahoo.vespa.config.*; +import com.yahoo.vespa.config.ConfigPayload; +import com.yahoo.vespa.config.ConfigTransformer; +import java.lang.reflect.Constructor; +import java.lang.reflect.Field; +import java.lang.reflect.InvocationTargetException; +import java.lang.reflect.Method; /** * @author gjoranv - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigInstanceUtil { /** diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigInterruptedException.java b/config/src/main/java/com/yahoo/config/subscription/ConfigInterruptedException.java index 050b7a81fa2..a7c2c2d9127 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigInterruptedException.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigInterruptedException.java @@ -4,10 +4,8 @@ package com.yahoo.config.subscription; /** * This exception is thrown when any blocking call within the Config API is interrupted. * @author Ulf Lilleengen - * @deprecated Will be removed in Vespa 8. Only for internal use. */ @SuppressWarnings("serial") -@Deprecated(forRemoval = true, since = "7") public class ConfigInterruptedException extends RuntimeException { public ConfigInterruptedException(Throwable cause) { super(cause); diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigSet.java b/config/src/main/java/com/yahoo/config/subscription/ConfigSet.java index 36d9047aa87..1b86db6b299 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigSet.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigSet.java @@ -1,19 +1,16 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.config.subscription; -import java.util.Map; -import java.util.concurrent.ConcurrentHashMap; - import com.yahoo.config.ConfigInstance; import com.yahoo.vespa.config.ConfigKey; +import java.util.Map; +import java.util.concurrent.ConcurrentHashMap; /** * Config source as a programmatically built set of {@link com.yahoo.config.ConfigInstance}s * * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigSet implements ConfigSource { private final Map<ConfigKey<?>, ConfigInstance.Builder> configs = new ConcurrentHashMap<>(); diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigSource.java b/config/src/main/java/com/yahoo/config/subscription/ConfigSource.java index b8abeb48c3d..5814a004216 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigSource.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigSource.java @@ -5,9 +5,7 @@ package com.yahoo.config.subscription; * A type of source of config * * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public interface ConfigSource { } diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigSourceSet.java b/config/src/main/java/com/yahoo/config/subscription/ConfigSourceSet.java index 37f91605a54..81aeda52b54 100755 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigSourceSet.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigSourceSet.java @@ -18,9 +18,7 @@ import static java.util.logging.Level.INFO; * upper/lower-casing and whitespaces. * * @author gjoranv - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigSourceSet implements ConfigSource { private static final Logger log = Logger.getLogger(ConfigSourceSet.class.getName()); diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigSubscriber.java b/config/src/main/java/com/yahoo/config/subscription/ConfigSubscriber.java index ee6ce37036c..1c516b37783 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigSubscriber.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigSubscriber.java @@ -28,9 +28,7 @@ import static java.util.stream.Collectors.toList; * {@link ConfigHandle} which {@link #subscribe(Class, String)} returned. * * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigSubscriber implements AutoCloseable { private static final Logger log = Logger.getLogger(ConfigSubscriber.class.getName()); diff --git a/config/src/main/java/com/yahoo/config/subscription/ConfigURI.java b/config/src/main/java/com/yahoo/config/subscription/ConfigURI.java index 46db3c6c83a..f0257457902 100644 --- a/config/src/main/java/com/yahoo/config/subscription/ConfigURI.java +++ b/config/src/main/java/com/yahoo/config/subscription/ConfigURI.java @@ -1,18 +1,15 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.config.subscription; -import java.io.File; - import com.yahoo.config.subscription.impl.JRTConfigRequester; +import java.io.File; /** * A Config URI is a class that can be used to encapsulate a config source and a config id into one * object to simplify parameter passing. * * @author Ulf Lilleengen - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class ConfigURI { private String configId; diff --git a/config/src/main/java/com/yahoo/config/subscription/DirSource.java b/config/src/main/java/com/yahoo/config/subscription/DirSource.java index 48d52bd49f3..fbd65ccf597 100644 --- a/config/src/main/java/com/yahoo/config/subscription/DirSource.java +++ b/config/src/main/java/com/yahoo/config/subscription/DirSource.java @@ -6,9 +6,7 @@ import java.io.File; /** * Source specifying config from a local directory * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class DirSource implements ConfigSource { private final File dir; diff --git a/config/src/main/java/com/yahoo/config/subscription/FileSource.java b/config/src/main/java/com/yahoo/config/subscription/FileSource.java index bbad7286b4c..2178c64cca3 100644 --- a/config/src/main/java/com/yahoo/config/subscription/FileSource.java +++ b/config/src/main/java/com/yahoo/config/subscription/FileSource.java @@ -7,9 +7,7 @@ import java.io.File; * Source specifying config from one local file * * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class FileSource implements ConfigSource { private final File file; diff --git a/config/src/main/java/com/yahoo/config/subscription/JarSource.java b/config/src/main/java/com/yahoo/config/subscription/JarSource.java index f39ee5e9262..c106823b52d 100644 --- a/config/src/main/java/com/yahoo/config/subscription/JarSource.java +++ b/config/src/main/java/com/yahoo/config/subscription/JarSource.java @@ -6,9 +6,7 @@ import java.util.jar.JarFile; /** * Source specifying config as a jar file entry * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class JarSource implements ConfigSource { private final String path; private final JarFile jarFile; diff --git a/config/src/main/java/com/yahoo/config/subscription/RawSource.java b/config/src/main/java/com/yahoo/config/subscription/RawSource.java index 28b249c6ece..8b73d2058f5 100644 --- a/config/src/main/java/com/yahoo/config/subscription/RawSource.java +++ b/config/src/main/java/com/yahoo/config/subscription/RawSource.java @@ -5,9 +5,7 @@ package com.yahoo.config.subscription; * Source specifying raw config, where payload is given programmatically * * @author Vegard Havdal - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class RawSource implements ConfigSource { public final String payload; diff --git a/config/src/main/java/com/yahoo/config/subscription/SubscriberClosedException.java b/config/src/main/java/com/yahoo/config/subscription/SubscriberClosedException.java index f7051ab1b38..1e6ebdbd752 100644 --- a/config/src/main/java/com/yahoo/config/subscription/SubscriberClosedException.java +++ b/config/src/main/java/com/yahoo/config/subscription/SubscriberClosedException.java @@ -5,7 +5,5 @@ package com.yahoo.config.subscription; * Thrown when {@link ConfigSubscriber} is closed * * @author bjorncs - * @deprecated Will be removed in Vespa 8. Only for internal use. */ -@Deprecated(forRemoval = true, since = "7") public class SubscriberClosedException extends RuntimeException {} diff --git a/config/src/main/java/com/yahoo/config/subscription/package-info.java b/config/src/main/java/com/yahoo/config/subscription/package-info.java index 6798809dee2..bda7649d184 100644 --- a/config/src/main/java/com/yahoo/config/subscription/package-info.java +++ b/config/src/main/java/com/yahoo/config/subscription/package-info.java @@ -1,10 +1,8 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. @ExportPackage -@PublicApi /** * Classes for subscribing to Vespa config. */ package com.yahoo.config.subscription; -import com.yahoo.api.annotations.PublicApi; import com.yahoo.osgi.annotation.ExportPackage; diff --git a/config/src/test/java/com/yahoo/config/subscription/ConfigGetterTest.java b/config/src/test/java/com/yahoo/config/subscription/ConfigGetterTest.java index 68cadb33ecd..fb116729640 100644 --- a/config/src/test/java/com/yahoo/config/subscription/ConfigGetterTest.java +++ b/config/src/test/java/com/yahoo/config/subscription/ConfigGetterTest.java @@ -2,11 +2,8 @@ package com.yahoo.config.subscription; import com.yahoo.foo.AppConfig; - import org.junit.Test; -import java.io.File; - import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; @@ -16,7 +13,7 @@ import static org.junit.Assert.assertTrue; * @author gjoranv */ public class ConfigGetterTest { - private ConfigSourceSet sourceSet = new ConfigSourceSet("config-getter-test"); + private final ConfigSourceSet sourceSet = new ConfigSourceSet("config-getter-test"); @Test public void testGetConfig() { @@ -41,13 +38,6 @@ public class ConfigGetterTest { } @Test - public void testGetFromRawSource() { - ConfigGetter<AppConfig> getter = new ConfigGetter<>(new RawSource("message \"one\""), AppConfig.class); - AppConfig config = getter.getConfig("test"); - assertEquals("one", config.message()); - } - - @Test public void testGetTwice() { ConfigGetter<AppConfig> getter = new ConfigGetter<>(AppConfig.class); AppConfig config = getter.getConfig("raw:message \"one\""); @@ -63,13 +53,6 @@ public class ConfigGetterTest { verifyFooValues(config); } - @Test - public void testGetFromFileSource() { - ConfigGetter<AppConfig> getter = new ConfigGetter<>(new FileSource(new File("src/test/resources/configs/foo/app.cfg")), AppConfig.class); - AppConfig config = getter.getConfig("test"); - verifyFooValues(config); - } - @Test public void testGetFromDir() { ConfigGetter<AppConfig> getter = new ConfigGetter<>(AppConfig.class); @@ -77,12 +60,6 @@ public class ConfigGetterTest { verifyFooValues(config); } - @Test - public void testGetFromDirSource() { - AppConfig config = ConfigGetter.getConfig(AppConfig.class, "test", new DirSource(new File("src/test/resources/configs/foo/"))); - verifyFooValues(config); - } - private void verifyFooValues(AppConfig config) { assertEquals("msg1", config.message()); assertEquals(3, config.times()); diff --git a/config/src/test/java/com/yahoo/config/subscription/ConfigSourceTest.java b/config/src/test/java/com/yahoo/config/subscription/ConfigSourceTest.java deleted file mode 100644 index e7baab407f6..00000000000 --- a/config/src/test/java/com/yahoo/config/subscription/ConfigSourceTest.java +++ /dev/null @@ -1,46 +0,0 @@ -// Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -package com.yahoo.config.subscription; - -import com.yahoo.foo.SimpletypesConfig; -import org.junit.Rule; -import org.junit.Test; -import org.junit.rules.TemporaryFolder; - -import java.io.File; -import java.io.IOException; - -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.fail; - -/** - * @author Ulf Lilleengen - */ -public class ConfigSourceTest { - - @Test(expected = IllegalArgumentException.class) - public void require_that_FileSource_throws_exception_on_invalid_file() { - new FileSource(new File("invalid")); - } - - @Test(expected = IllegalArgumentException.class) - public void require_that_DirSource_throws_exception_on_invalid_dir() { - new DirSource(new File("invalid")); - } - - @Rule - public TemporaryFolder tmpDir = new TemporaryFolder(); - - @Test - public void require_that_DirSource_throws_exception_on_missing_file() throws IOException { - File folder = tmpDir.newFolder(); - DirSource dirSource = new DirSource(folder); - try { - ConfigGetter.getConfig(SimpletypesConfig.class, "dir:" + tmpDir, dirSource); - fail(); - } catch (IllegalArgumentException e) { - assertEquals("Could not find a config file for '" + SimpletypesConfig.getDefName() + "' in '" + folder + "/'", - e.getMessage()); - } - } - -} diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileServer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileServer.java index 15bd898c053..88405b3eef9 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileServer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileServer.java @@ -232,7 +232,6 @@ public class FileServer { } private static ConnectionPool createConnectionPool(List<String> configServers, Supervisor supervisor) { - @SuppressWarnings("removal") // TODO Vespa 8: remove ConfigSourceSet configSourceSet = new ConfigSourceSet(configServers); if (configServers.size() == 0) return FileDownloader.emptyConnectionPool(); diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java index ccb17146df0..b2915bc74fb 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java @@ -96,7 +96,6 @@ public class ApplicationPackageMaintainer extends ConfigServerMaintainer { File downloadDirectory, Supervisor supervisor) { List<String> otherConfigServersInCluster = getOtherConfigServersInCluster(configserverConfig); - @SuppressWarnings("removal") // TODO Vespa 8: remove ConfigSourceSet configSourceSet = new ConfigSourceSet(otherConfigServersInCluster); ConnectionPool connectionPool = (otherConfigServersInCluster.isEmpty()) diff --git a/container-core/src/main/java/com/yahoo/container/core/config/testutil/HandlersConfigurerTestWrapper.java b/container-core/src/main/java/com/yahoo/container/core/config/testutil/HandlersConfigurerTestWrapper.java index 96a479ac7ad..62d36ec34ec 100644 --- a/container-core/src/main/java/com/yahoo/container/core/config/testutil/HandlersConfigurerTestWrapper.java +++ b/container-core/src/main/java/com/yahoo/container/core/config/testutil/HandlersConfigurerTestWrapper.java @@ -39,7 +39,6 @@ import java.util.concurrent.Executors; */ public class HandlersConfigurerTestWrapper { - @SuppressWarnings("removal") // TODO Vespa 8: remove private final ConfigSourceSet configSources = new ConfigSourceSet(this.getClass().getSimpleName() + ": " + new Random().nextLong()); private final HandlersConfigurerDi configurer; diff --git a/container-core/src/main/java/com/yahoo/container/di/CloudSubscriber.java b/container-core/src/main/java/com/yahoo/container/di/CloudSubscriber.java index 2acbec2e388..5cbb009d54b 100644 --- a/container-core/src/main/java/com/yahoo/container/di/CloudSubscriber.java +++ b/container-core/src/main/java/com/yahoo/container/di/CloudSubscriber.java @@ -7,11 +7,9 @@ import com.yahoo.config.subscription.ConfigSource; import com.yahoo.config.subscription.ConfigSubscriber; import com.yahoo.container.di.config.Subscriber; import com.yahoo.vespa.config.ConfigKey; - import java.util.HashMap; import java.util.Map; import java.util.Set; -import java.util.logging.Level; import java.util.logging.Logger; import static java.util.logging.Level.FINE; @@ -21,7 +19,6 @@ import static java.util.logging.Level.FINE; * @author Tony Vaagenes * @author ollivir */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class CloudSubscriber implements Subscriber { private static final Logger log = Logger.getLogger(CloudSubscriber.class.getName()); diff --git a/container-core/src/main/java/com/yahoo/container/di/CloudSubscriberFactory.java b/container-core/src/main/java/com/yahoo/container/di/CloudSubscriberFactory.java index a58baa5b2bd..a6327b01e21 100644 --- a/container-core/src/main/java/com/yahoo/container/di/CloudSubscriberFactory.java +++ b/container-core/src/main/java/com/yahoo/container/di/CloudSubscriberFactory.java @@ -20,7 +20,6 @@ import java.util.WeakHashMap; * @author Tony Vaagenes * @author ollivir */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class CloudSubscriberFactory implements SubscriberFactory { private final ConfigSource configSource; diff --git a/container-core/src/main/java/com/yahoo/container/di/Container.java b/container-core/src/main/java/com/yahoo/container/di/Container.java index 9977813b533..68dda5fbc4c 100644 --- a/container-core/src/main/java/com/yahoo/container/di/Container.java +++ b/container-core/src/main/java/com/yahoo/container/di/Container.java @@ -237,7 +237,6 @@ public class Container { } } - @SuppressWarnings("removal") // TODO Vespa 8: remove private void invalidateGeneration(long generation, Throwable cause) { leastGeneration = Math.max(retriever.getComponentsGeneration(), retriever.getBootstrapGeneration()) + 1; if (!(cause instanceof InterruptedException) && !(cause instanceof ConfigInterruptedException) && !(cause instanceof SubscriberClosedException)) { diff --git a/container-core/src/test/java/com/yahoo/container/di/DirConfigSource.java b/container-core/src/test/java/com/yahoo/container/di/DirConfigSource.java index 34119163d2f..d459070e2bd 100644 --- a/container-core/src/test/java/com/yahoo/container/di/DirConfigSource.java +++ b/container-core/src/test/java/com/yahoo/container/di/DirConfigSource.java @@ -16,7 +16,6 @@ import java.util.Random; * @author gjoranv * @author ollivir */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class DirConfigSource { private final TemporaryFolder tempFolder = createTemporaryFolder(); public final ConfigSource configSource; diff --git a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java index 71fe4f8d81c..7782e54dc90 100644 --- a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java +++ b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java @@ -308,7 +308,6 @@ public final class ConfiguredApplication implements Application { return builder; } - @SuppressWarnings("removal") // TODO Vespa 8: remove private void doReconfigurationLoop() { while (!shutdownReconfiguration) { try { diff --git a/docproc/src/main/java/com/yahoo/docproc/proxy/SchemaMap.java b/docproc/src/main/java/com/yahoo/docproc/proxy/SchemaMap.java index ffe89e5a405..8a544dea221 100644 --- a/docproc/src/main/java/com/yahoo/docproc/proxy/SchemaMap.java +++ b/docproc/src/main/java/com/yahoo/docproc/proxy/SchemaMap.java @@ -16,7 +16,6 @@ import java.util.Map.Entry; * * @author vegardh */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class SchemaMap implements ConfigSubscriber.SingleSubscriber<SchemamappingConfig> { /** Map key. Doctype can be null, not the others. */ diff --git a/document/src/main/java/com/yahoo/document/DocumentTypeManager.java b/document/src/main/java/com/yahoo/document/DocumentTypeManager.java index 277f0927be1..880d8f3fab0 100644 --- a/document/src/main/java/com/yahoo/document/DocumentTypeManager.java +++ b/document/src/main/java/com/yahoo/document/DocumentTypeManager.java @@ -40,7 +40,6 @@ public class DocumentTypeManager { private final static Logger log = Logger.getLogger(DocumentTypeManager.class.getName()); - @SuppressWarnings("removal") // TODO Vespa 8: remove private ConfigSubscriber subscriber; // *Configured data types* (not built-in/primitive) indexed by their id diff --git a/document/src/main/java/com/yahoo/document/DocumentTypeManagerConfigurer.java b/document/src/main/java/com/yahoo/document/DocumentTypeManagerConfigurer.java index 3ecf7503013..17ecd2b9659 100644 --- a/document/src/main/java/com/yahoo/document/DocumentTypeManagerConfigurer.java +++ b/document/src/main/java/com/yahoo/document/DocumentTypeManagerConfigurer.java @@ -7,24 +7,23 @@ import com.yahoo.document.annotation.AnnotationReferenceDataType; import com.yahoo.document.annotation.AnnotationType; import com.yahoo.document.config.DocumentmanagerConfig; import com.yahoo.document.internal.GeoPosType; -import java.util.logging.Level; +import com.yahoo.tensor.TensorType; import java.util.ArrayList; import java.util.Collection; import java.util.HashMap; -import java.util.Map; import java.util.HashSet; +import java.util.Map; import java.util.Set; +import java.util.function.Supplier; +import java.util.logging.Level; import java.util.logging.Logger; import java.util.stream.Collectors; -import java.util.function.Supplier; -import com.yahoo.tensor.TensorType; /** * Configures the Vespa document manager from a config id. * * @author Einar M R Rosenvinge */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class DocumentTypeManagerConfigurer implements ConfigSubscriber.SingleSubscriber<DocumentmanagerConfig> { private final static Logger log = Logger.getLogger(DocumentTypeManagerConfigurer.class.getName()); diff --git a/documentapi/src/main/java/com/yahoo/documentapi/DocumentAccess.java b/documentapi/src/main/java/com/yahoo/documentapi/DocumentAccess.java index c8f4c3900e1..9a6e634f071 100644 --- a/documentapi/src/main/java/com/yahoo/documentapi/DocumentAccess.java +++ b/documentapi/src/main/java/com/yahoo/documentapi/DocumentAccess.java @@ -47,7 +47,6 @@ import com.yahoo.documentapi.messagebus.MessageBusDocumentAccess; public abstract class DocumentAccess { private final DocumentTypeManager documentTypeManager; - @SuppressWarnings("removal") // TODO Vespa 8: remove private final ConfigSubscriber documentTypeConfigSubscriber; /** diff --git a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java index a785438da80..fbc8c9deb8e 100755 --- a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java +++ b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/DocumentRouteSelectorPolicy.java @@ -23,11 +23,10 @@ import java.util.logging.Logger; * * @author Simon Thoresen Hult */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class DocumentRouteSelectorPolicy implements DocumentProtocolRoutingPolicy, ConfigSubscriber.SingleSubscriber<DocumentrouteselectorpolicyConfig> { - private static Logger log = Logger.getLogger(DocumentRouteSelectorPolicy.class.getName()); + private static final Logger log = Logger.getLogger(DocumentRouteSelectorPolicy.class.getName()); private Map<String, DocumentSelector> config; private String error = "Not configured."; private ConfigSubscriber subscriber; diff --git a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/MessageTypePolicy.java b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/MessageTypePolicy.java index 095d1f433d2..57cd70cf4b4 100644 --- a/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/MessageTypePolicy.java +++ b/documentapi/src/main/java/com/yahoo/documentapi/messagebus/protocol/MessageTypePolicy.java @@ -15,7 +15,6 @@ import static java.util.stream.Collectors.toUnmodifiableMap; /** * @author baldersheim */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class MessageTypePolicy implements DocumentProtocolRoutingPolicy, ConfigSubscriber.SingleSubscriber<MessagetyperouteselectorpolicyConfig> { private final AtomicReference<Map<Integer, Route>> configRef = new AtomicReference<>(); diff --git a/fileacquirer/src/main/java/com/yahoo/filedistribution/fileacquirer/FileAcquirerImpl.java b/fileacquirer/src/main/java/com/yahoo/filedistribution/fileacquirer/FileAcquirerImpl.java index 5ab2d3d7246..c85a24db9bb 100644 --- a/fileacquirer/src/main/java/com/yahoo/filedistribution/fileacquirer/FileAcquirerImpl.java +++ b/fileacquirer/src/main/java/com/yahoo/filedistribution/fileacquirer/FileAcquirerImpl.java @@ -38,10 +38,8 @@ class FileAcquirerImpl implements FileAcquirer { private final Supervisor supervisor = new Supervisor(new Transport("fileaquirer")); - @SuppressWarnings("removal") // TODO Vespa 8: remove private final ConfigSubscriber configSubscriber; - @SuppressWarnings("removal") // TODO Vespa 8: remove private class Connection implements ConfigSubscriber.SingleSubscriber<FiledistributorrpcConfig> { private final Lock targetLock = new ReentrantLock(); private Target target; @@ -125,7 +123,6 @@ class FileAcquirerImpl implements FileAcquirer { } } - @SuppressWarnings("removal") // TODO Vespa 8: remove public FileAcquirerImpl(String configId) { configSubscriber = new ConfigSubscriber(); configSubscriber.subscribe(connection, FiledistributorrpcConfig.class, configId); diff --git a/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileDistributionConnectionPool.java b/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileDistributionConnectionPool.java index c292080a0e5..3a03e6a87d5 100644 --- a/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileDistributionConnectionPool.java +++ b/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileDistributionConnectionPool.java @@ -22,7 +22,6 @@ import java.util.List; */ public class FileDistributionConnectionPool extends JRTConnectionPool { - @SuppressWarnings("removal") // TODO Vespa 8: remove public FileDistributionConnectionPool(ConfigSourceSet sourceSet, Supervisor supervisor) { super(sourceSet, supervisor); } diff --git a/messagebus/src/main/java/com/yahoo/messagebus/ConfigAgent.java b/messagebus/src/main/java/com/yahoo/messagebus/ConfigAgent.java index 10ebb2fc1ae..4db97f0c083 100755 --- a/messagebus/src/main/java/com/yahoo/messagebus/ConfigAgent.java +++ b/messagebus/src/main/java/com/yahoo/messagebus/ConfigAgent.java @@ -14,7 +14,6 @@ import com.yahoo.messagebus.routing.RoutingTableSpec; * * @author Simon Thoresen Hult */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class ConfigAgent implements ConfigSubscriber.SingleSubscriber<MessagebusConfig>{ private final ConfigURI configURI; diff --git a/messagebus/src/main/java/com/yahoo/messagebus/network/rpc/SlobrokConfigSubscriber.java b/messagebus/src/main/java/com/yahoo/messagebus/network/rpc/SlobrokConfigSubscriber.java index 02472d8f31f..903a31d3f3a 100755 --- a/messagebus/src/main/java/com/yahoo/messagebus/network/rpc/SlobrokConfigSubscriber.java +++ b/messagebus/src/main/java/com/yahoo/messagebus/network/rpc/SlobrokConfigSubscriber.java @@ -13,7 +13,6 @@ import java.util.logging.Logger; * * @author Simon Thoresen Hult */ -@SuppressWarnings("removal") // TODO Vespa 8: remove public class SlobrokConfigSubscriber implements ConfigSubscriber.SingleSubscriber<SlobroksConfig>{ private static final Logger log = Logger.getLogger(SlobrokConfigSubscriber.class.getName()); diff --git a/messagebus/src/test/java/com/yahoo/messagebus/ConfigAgentTestCase.java b/messagebus/src/test/java/com/yahoo/messagebus/ConfigAgentTestCase.java index 287529ae449..a32175ff414 100755 --- a/messagebus/src/test/java/com/yahoo/messagebus/ConfigAgentTestCase.java +++ b/messagebus/src/test/java/com/yahoo/messagebus/ConfigAgentTestCase.java @@ -22,7 +22,6 @@ public class ConfigAgentTestCase { @Rule public TemporaryFolder tmpFolder = new TemporaryFolder(); - @SuppressWarnings("removal") // TODO Vespa 8: remove @Test public void testRoutingConfig() throws InterruptedException { LocalHandler handler = new LocalHandler(); diff --git a/standalone-container/src/main/java/com/yahoo/container/standalone/StandaloneSubscriberFactory.java b/standalone-container/src/main/java/com/yahoo/container/standalone/StandaloneSubscriberFactory.java index 5555b3b4077..b1d3f1a6e2c 100644 --- a/standalone-container/src/main/java/com/yahoo/container/standalone/StandaloneSubscriberFactory.java +++ b/standalone-container/src/main/java/com/yahoo/container/standalone/StandaloneSubscriberFactory.java @@ -57,7 +57,6 @@ public class StandaloneSubscriberFactory implements SubscriberFactory { return ret; } - @SuppressWarnings("removal") // TODO Vespa 8: remove @Override public long waitNextGeneration(boolean isInitializing) { generation++; diff --git a/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java b/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java index b6455a7703c..04a577db554 100644 --- a/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java +++ b/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java @@ -39,7 +39,6 @@ public class Distribution { private final boolean distributorAutoOwnershipTransferOnWholeGroupDown; } - @SuppressWarnings("removal") // TODO Vespa 8: remove private ConfigSubscriber configSub; private final AtomicReference<Config> config = new AtomicReference<>(new Config(null, 1, false)); @@ -61,7 +60,6 @@ public class Distribution { return p; } - @SuppressWarnings("removal") // TODO Vespa 8: remove // NOTE: keep in sync with the below private ConfigSubscriber.SingleSubscriber<StorDistributionConfig> configSubscriber = config -> { try { @@ -148,7 +146,6 @@ public class Distribution { } } - @SuppressWarnings("removal") // TODO Vespa 8: remove public Distribution(String configId) { try { configSub = new ConfigSubscriber(); |