diff options
author | Jon Bratseth <bratseth@gmail.com> | 2021-01-14 12:29:18 +0100 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2021-01-14 12:29:18 +0100 |
commit | 16e92fc5817b725925f173be45309ab9371cf46d (patch) | |
tree | e8711ae34743c3f0a429fb6732512dc2d66aff61 /config-model | |
parent | c1084399767a5aaf504229ac9c485c696a0832d3 (diff) |
Remove unused provider yca fields
Diffstat (limited to 'config-model')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/chains/search/DomProviderBuilder.java | 22 |
1 files changed, 4 insertions, 18 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/chains/search/DomProviderBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/chains/search/DomProviderBuilder.java index 534b07339ec..7fdf7d6d771 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/chains/search/DomProviderBuilder.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/builder/xml/dom/chains/search/DomProviderBuilder.java @@ -36,6 +36,7 @@ public class DomProviderBuilder extends DomGenericTargetBuilder<Provider> { * Retrieves all possible provider specific parameters */ private static class ProviderReader { + final String type; final String path; final Double cacheWeight; @@ -45,10 +46,6 @@ public class DomProviderBuilder extends DomGenericTargetBuilder<Provider> { final Double connectionPoolTimeout; final String clusterName; final List<Node> nodes; - final String certificateApplicationId; - final Integer certificateTtl; - final Integer certificateRetryWait; - final Node certificateProxy; // Just re-using the Node class, as it matches our needs final Integer cacheSizeMB; ProviderReader(Element providerElement) { @@ -62,10 +59,6 @@ public class DomProviderBuilder extends DomGenericTargetBuilder<Provider> { connectionPoolTimeout = readConnectionPoolTimeout(providerElement); retries = readRetries(providerElement); nodes = readNodes(providerElement); - certificateApplicationId = readCertificateApplicationId(providerElement); - certificateTtl = readCertificateTtl(providerElement); - certificateRetryWait = readCertificateRetryWait(providerElement); - certificateProxy = readCertificateProxy(providerElement); } @@ -172,12 +165,6 @@ public class DomProviderBuilder extends DomGenericTargetBuilder<Provider> { ChainSpecification specWithoutInnerComponents) { ProviderReader providerReader = new ProviderReader(providerElement); - if (providerReader.certificateApplicationId == null && providerReader.certificateProxy != null) { - throw new IllegalArgumentException( - "Provider '" + specWithoutInnerComponents.componentId + - "' must have a certificate application ID, since a certificate store proxy is given"); - } - FederationOptions federationOptions = readFederationOptions(providerElement); Provider provider = buildProvider(specWithoutInnerComponents, providerReader, federationOptions); @@ -220,12 +207,11 @@ public class DomProviderBuilder extends DomGenericTargetBuilder<Provider> { try { ensureEmpty(specWithoutInnerSearchers.componentId, providerReader.cacheWeight, providerReader.path, providerReader.nodes, providerReader.readTimeout, providerReader.connectionTimeout, providerReader.connectionPoolTimeout, - providerReader.retries, providerReader.certificateApplicationId, providerReader.certificateTtl, - providerReader.certificateRetryWait, providerReader.certificateProxy); + providerReader.retries); return new LocalProvider(specWithoutInnerSearchers, - federationOptions, - new LocalProviderSpec(providerReader.clusterName, providerReader.cacheSizeMB)); + federationOptions, + new LocalProviderSpec(providerReader.clusterName, providerReader.cacheSizeMB)); } catch (Exception e) { throw new RuntimeException("Failed creating local provider " + specWithoutInnerSearchers.componentId, e); } |