diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2020-10-28 17:38:04 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2020-10-28 17:38:04 +0000 |
commit | da3bcaee0309daad4ed9ab6f7f2bd56986302e35 (patch) | |
tree | 25e0881dbd6a4559bd4b2862f5008a13604d8db3 /config-model | |
parent | 02785bc255415bb26aaf57be10b4261e2ce65bc8 (diff) |
GC unused content bucket btree flag
Diffstat (limited to 'config-model')
4 files changed, 6 insertions, 34 deletions
diff --git a/config-model/src/main/java/com/yahoo/config/model/deploy/TestProperties.java b/config-model/src/main/java/com/yahoo/config/model/deploy/TestProperties.java index 7628af85b52..e1e2652b987 100644 --- a/config-model/src/main/java/com/yahoo/config/model/deploy/TestProperties.java +++ b/config-model/src/main/java/com/yahoo/config/model/deploy/TestProperties.java @@ -35,7 +35,6 @@ public class TestProperties implements ModelContext.Properties { private Zone zone; private final Set<ContainerEndpoint> endpoints = Collections.emptySet(); private boolean useDedicatedNodeForLogserver = false; - private boolean useContentNodeBtreeDb = false; private boolean useThreePhaseUpdates = false; private boolean useDirectStorageApiRpc = false; private boolean useFastValueTensorImplementation = false; @@ -72,7 +71,6 @@ public class TestProperties implements ModelContext.Properties { @Override public boolean useDedicatedNodeForLogserver() { return useDedicatedNodeForLogserver; } @Override public Optional<EndpointCertificateSecrets> endpointCertificateSecrets() { return endpointCertificateSecrets; } @Override public double defaultTermwiseLimit() { return defaultTermwiseLimit; } - @Override public boolean useContentNodeBtreeDb() { return useContentNodeBtreeDb; } @Override public boolean useThreePhaseUpdates() { return useThreePhaseUpdates; } @Override public boolean useDirectStorageApiRpc() { return useDirectStorageApiRpc; } @Override public boolean useFastValueTensorImplementation() { return useFastValueTensorImplementation; } @@ -111,11 +109,6 @@ public class TestProperties implements ModelContext.Properties { return this; } - public TestProperties setUseContentNodeBtreeDB(boolean useBtreeDb) { - useContentNodeBtreeDb = useBtreeDb; - return this; - } - public TestProperties setUseThreePhaseUpdates(boolean useThreePhaseUpdates) { this.useThreePhaseUpdates = useThreePhaseUpdates; return this; diff --git a/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorServerProducer.java b/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorServerProducer.java index 9e4c9bde1e4..cb8221e3ad6 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorServerProducer.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorServerProducer.java @@ -10,36 +10,33 @@ import com.yahoo.vespa.model.builder.xml.dom.ModelElement; */ public class StorServerProducer implements StorServerConfig.Producer { public static class Builder { - StorServerProducer build(ModelElement element, boolean useBtreeDatabase) { + StorServerProducer build(ModelElement element) { ModelElement tuning = element.child("tuning"); if (tuning == null) { - return new StorServerProducer(ContentCluster.getClusterId(element), null, null, useBtreeDatabase); + return new StorServerProducer(ContentCluster.getClusterId(element), null, null); } ModelElement merges = tuning.child("merges"); if (merges == null) { - return new StorServerProducer(ContentCluster.getClusterId(element), null, null, useBtreeDatabase); + return new StorServerProducer(ContentCluster.getClusterId(element), null, null); } return new StorServerProducer(ContentCluster.getClusterId(element), merges.integerAttribute("max-per-node"), - merges.integerAttribute("max-queue-size"), - useBtreeDatabase); + merges.integerAttribute("max-queue-size")); } } private final String clusterName; private final Integer maxMergesPerNode; private final Integer queueSize; - private final boolean useBtreeDatabase; public StorServerProducer(String clusterName, Integer maxMergesPerNode, - Integer queueSize, boolean useBtreeDatabase) { + Integer queueSize) { this.clusterName = clusterName; this.maxMergesPerNode = maxMergesPerNode; this.queueSize = queueSize; - this.useBtreeDatabase = useBtreeDatabase; } @Override @@ -56,6 +53,5 @@ public class StorServerProducer implements StorServerConfig.Producer { if (queueSize != null) { builder.max_merge_queue_size(queueSize); } - builder.use_content_node_btree_bucket_db(useBtreeDatabase); } } diff --git a/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java index bacb22b0b89..b9fd26c45ff 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java @@ -33,13 +33,12 @@ public class StorageCluster extends AbstractConfigProducer<StorageNode> protected StorageCluster doBuild(DeployState deployState, AbstractConfigProducer ancestor, Element producerSpec) { final ModelElement clusterElem = new ModelElement(producerSpec); final ContentCluster cluster = (ContentCluster)ancestor; - boolean useContentNodeBtreeDb = deployState.getProperties().useContentNodeBtreeDb(); return new StorageCluster(ancestor, ContentCluster.getClusterId(clusterElem), new FileStorProducer.Builder().build(deployState.getProperties(), cluster, clusterElem), new IntegrityCheckerProducer.Builder().build(cluster, clusterElem), - new StorServerProducer.Builder().build(clusterElem, useContentNodeBtreeDb), + new StorServerProducer.Builder().build(clusterElem), new StorVisitorProducer.Builder().build(clusterElem), new PersistenceProducer.Builder().build(clusterElem)); } diff --git a/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java b/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java index 2ec82697f23..92e9ca43193 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java @@ -950,22 +950,6 @@ public class ContentClusterTest extends ContentBaseTest { verifyTopKProbabilityPropertiesControl(); } - private boolean resolveContentNodeBtreeDbConfigWithFeatureFlag(boolean flagEnabledBtreeDb) { - VespaModel model = createEnd2EndOneNode(new TestProperties().setUseContentNodeBtreeDB(flagEnabledBtreeDb)); - - ContentCluster cc = model.getContentClusters().get("storage"); - var builder = new StorServerConfig.Builder(); - cc.getStorageNodes().getConfig(builder); - - return (new StorServerConfig(builder)).use_content_node_btree_bucket_db(); - } - - @Test - public void default_content_node_btree_usage_controlled_by_properties() { - assertFalse(resolveContentNodeBtreeDbConfigWithFeatureFlag(false)); - assertTrue(resolveContentNodeBtreeDbConfigWithFeatureFlag(true)); - } - private boolean resolveThreePhaseUpdateConfigWithFeatureFlag(boolean flagEnableThreePhase) { VespaModel model = createEnd2EndOneNode(new TestProperties().setUseThreePhaseUpdates(flagEnableThreePhase)); |