diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2018-01-29 16:06:59 +0100 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2018-01-29 16:06:59 +0100 |
commit | 6fa6ba840c832689ce92628923fc646c59fe26f1 (patch) | |
tree | cc4f1fe7219c326f37b4733aae95c8dff93ddea9 /config-model/src | |
parent | c90710ca371546d34023470f08929b4cd73b5ff9 (diff) |
Test that summary cache is given 5% of memory unless specified explicit.
Diffstat (limited to 'config-model/src')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/search/NodeFlavorTuning.java | 6 | ||||
-rw-r--r-- | config-model/src/test/java/com/yahoo/vespa/model/search/NodeFlavorTuningTest.java | 6 |
2 files changed, 12 insertions, 0 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/search/NodeFlavorTuning.java b/config-model/src/main/java/com/yahoo/vespa/model/search/NodeFlavorTuning.java index e4806cb2353..f6f64eba482 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/search/NodeFlavorTuning.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/search/NodeFlavorTuning.java @@ -30,6 +30,12 @@ public class NodeFlavorTuning implements ProtonConfig.Producer { tuneFlushStrategyMemoryLimits(builder.flush.memory); tuneFlushStrategyTlsSize(builder.flush.memory); tuneSummaryReadIo(builder.summary.read); + tuneSummaryCache(builder.summary.cache); + } + + private void tuneSummaryCache(ProtonConfig.Summary.Cache.Builder builder) { + long memoryLimitBytes = (long) ((nodeFlavor.getMinMainMemoryAvailableGb() * 0.05) * GB); + builder.maxbytes(memoryLimitBytes); } private void setHwInfo(ProtonConfig.Builder builder) { diff --git a/config-model/src/test/java/com/yahoo/vespa/model/search/NodeFlavorTuningTest.java b/config-model/src/test/java/com/yahoo/vespa/model/search/NodeFlavorTuningTest.java index 3d62670bb52..3019b35cd2e 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/search/NodeFlavorTuningTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/search/NodeFlavorTuningTest.java @@ -87,6 +87,12 @@ public class NodeFlavorTuningTest { } @Test + public void require_that_summary_cache_max_bytes_is_set_based_on_memory() { + assertEquals(1*GB/20, configFromMemorySetting(1).summary().cache().maxbytes()); + assertEquals(256*GB/20, configFromMemorySetting(256).summary().cache().maxbytes()); + } + + @Test public void require_that_docker_node_is_tagged_with_shared_disk() { assertSharedDisk(true, true); assertSharedDisk(false, false); |