From a53696877acab6e08168ffd60b3cbfa19b0658e3 Mon Sep 17 00:00:00 2001 From: Geir Storli Date: Wed, 18 May 2022 09:57:31 +0000 Subject: Remove feeding concurrency setting per document db that is no longer used. --- .../vespa/model/content/ContentSearchCluster.java | 9 +---- .../search/test/DocumentDatabaseTestCase.java | 43 ++++++++++------------ 2 files changed, 20 insertions(+), 32 deletions(-) (limited to 'config-model') diff --git a/config-model/src/main/java/com/yahoo/vespa/model/content/ContentSearchCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/content/ContentSearchCluster.java index 8b9f32fa135..2d01f9d903e 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/content/ContentSearchCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/content/ContentSearchCluster.java @@ -381,18 +381,11 @@ public class ContentSearchCluster extends AbstractConfigProducer hasAnyNonIndexedCluster = true; ddbB.inputdoctypename(type.getFullName().getName()) .configid(findStreamingCluster(docTypeName).get().getDocumentDBConfigId()) - .mode(ProtonConfig.Documentdb.Mode.Enum.STREAMING) - .feeding.concurrency(0.0); + .mode(ProtonConfig.Documentdb.Mode.Enum.STREAMING); } else if (hasIndexingModeIndexed(type)) { getIndexed().fillDocumentDBConfig(type.getFullName().getName(), ddbB); - if (tuning != null && tuning.searchNode != null && tuning.searchNode.feeding != null) { - ddbB.feeding.concurrency(tuning.searchNode.feeding.concurrency); - } else { - ddbB.feeding.concurrency(defaultFeedConcurrency); - } } else { hasAnyNonIndexedCluster = true; - ddbB.feeding.concurrency(0.0); ddbB.mode(ProtonConfig.Documentdb.Mode.Enum.STORE_ONLY); } if (globalDocType) { diff --git a/config-model/src/test/java/com/yahoo/vespa/model/search/test/DocumentDatabaseTestCase.java b/config-model/src/test/java/com/yahoo/vespa/model/search/test/DocumentDatabaseTestCase.java index 059c1ceb208..d541a6422e7 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/search/test/DocumentDatabaseTestCase.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/search/test/DocumentDatabaseTestCase.java @@ -42,21 +42,21 @@ public class DocumentDatabaseTestCase { @Test public void requireThatConcurrencyIsReflectedCorrectlyForDefault() { - verifyConcurrency("index", "", 0.50, 0.50); - verifyConcurrency("streaming", "", 1.0, 0.0); - verifyConcurrency("store-only", "", 1.0, 0.0); + verifyConcurrency("index", "", 0.50); + verifyConcurrency("streaming", "", 1.0); + verifyConcurrency("store-only", "", 1.0); } @Test public void requireThatFeatureFlagConcurrencyIsReflectedCorrectlyForDefault() { - verifyConcurrency("index", "", 0.30, 0.30, 0.3); - verifyConcurrency("streaming", "", 0.6, 0.0, 0.3); - verifyConcurrency("store-only", "", 0.8, 0.0, 0.4); + verifyConcurrency("index", "", 0.30, 0.3); + verifyConcurrency("streaming", "", 0.6, 0.3); + verifyConcurrency("store-only", "", 0.8, 0.4); } @Test public void requireThatMixedModeConcurrencyIsReflectedCorrectlyForDefault() { - verifyConcurrency(Arrays.asList(DocType.create("a", "index"), DocType.create("b", "streaming")), "", 1.0, Arrays.asList(0.50, 0.0)); + verifyConcurrency(Arrays.asList(DocType.create("a", "index"), DocType.create("b", "streaming")), "", 1.0); } @Test @@ -64,7 +64,7 @@ public class DocumentDatabaseTestCase { String feedTuning = "" + " 0.7" + "\n"; - verifyConcurrency(Arrays.asList(DocType.create("a", "index"), DocType.create("b", "streaming")), feedTuning, 0.7, Arrays.asList(0.7, 0.0)); + verifyConcurrency(Arrays.asList(DocType.create("a", "index"), DocType.create("b", "streaming")), feedTuning, 0.7); } @Test @@ -72,25 +72,24 @@ public class DocumentDatabaseTestCase { String feedTuning = "" + " 0.7" + "\n"; - verifyConcurrency("index", feedTuning, 0.7, 0.7); - verifyConcurrency("streaming", feedTuning, 0.7, 0.0); - verifyConcurrency("store-only", feedTuning, 0.7, 0.0); + verifyConcurrency("index", feedTuning, 0.7); + verifyConcurrency("streaming", feedTuning, 0.7); + verifyConcurrency("store-only", feedTuning, 0.7); } - private void verifyConcurrency(String mode, String xmlTuning, double global, double local, double featureFlagConcurrency) { - verifyConcurrency(Arrays.asList(DocType.create("a", mode)), xmlTuning, global, Arrays.asList(local), featureFlagConcurrency); + private void verifyConcurrency(String mode, String xmlTuning, double expectedConcurrency, double featureFlagConcurrency) { + verifyConcurrency(Arrays.asList(DocType.create("a", mode)), xmlTuning, expectedConcurrency, featureFlagConcurrency); } - private void verifyConcurrency(String mode, String xmlTuning, double global, double local) { - verifyConcurrency(Arrays.asList(DocType.create("a", mode)), xmlTuning, global, Arrays.asList(local), null); + private void verifyConcurrency(String mode, String xmlTuning, double expectedConcurrency) { + verifyConcurrency(Arrays.asList(DocType.create("a", mode)), xmlTuning, expectedConcurrency, null); } - private void verifyConcurrency(List nameAndModes, String xmlTuning, double global, List local) { - verifyConcurrency(nameAndModes, xmlTuning, global, local, null); + private void verifyConcurrency(List nameAndModes, String xmlTuning, double expectedConcurrency) { + verifyConcurrency(nameAndModes, xmlTuning, expectedConcurrency, null); } - private void verifyConcurrency(List nameAndModes, String xmlTuning, double global, List local, Double featureFlagConcurrency) { - assertEquals(nameAndModes.size(), local.size()); + private void verifyConcurrency(List nameAndModes, String xmlTuning, double expectedConcurrency, Double featureFlagConcurrency) { TestProperties properties = new TestProperties(); if (featureFlagConcurrency != null) { properties.setFeedConcurrency(featureFlagConcurrency); @@ -99,11 +98,7 @@ public class DocumentDatabaseTestCase { VespaModel model = tester.createModel(nameAndModes, xmlTuning, new DeployState.Builder().properties(properties)); ContentSearchCluster contentSearchCluster = model.getContentClusters().get("test").getSearch(); ProtonConfig proton = tester.getProtonConfig(contentSearchCluster); - assertEquals(global, proton.feeding().concurrency(), SMALL); - assertEquals(local.size(), proton.documentdb().size()); - for (int i = 0; i < local.size(); i++) { - assertEquals(local.get(i), proton.documentdb(i).feeding().concurrency(), SMALL); - } + assertEquals(expectedConcurrency, proton.feeding().concurrency(), SMALL); } @Test -- cgit v1.2.3