diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2018-10-12 14:34:47 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2018-10-12 14:34:47 +0200 |
commit | 3c202499a84914d66fadde9fbc0a8ac2fdeb96e3 (patch) | |
tree | a033785c375650b17dedca6e22b7ef287818e7cf | |
parent | f2e0e1d72d30121cf1458246dacce3f85f84ddc0 (diff) |
Do not warn until Vespa 7
-rw-r--r-- | config-model/src/main/java/com/yahoo/searchdefinition/processing/SummaryDiskAccessValidator.java | 9 | ||||
-rw-r--r-- | config-model/src/test/java/com/yahoo/searchdefinition/SummaryTestCase.java | 2 |
2 files changed, 6 insertions, 5 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/processing/SummaryDiskAccessValidator.java b/config-model/src/main/java/com/yahoo/searchdefinition/processing/SummaryDiskAccessValidator.java index 8c0c6a74037..d30006a753e 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/processing/SummaryDiskAccessValidator.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/processing/SummaryDiskAccessValidator.java @@ -42,10 +42,11 @@ public class SummaryDiskAccessValidator extends Processor { throw new IllegalArgumentException(summaryField + " in " + summary + " references " + source + ", but this field does not exist"); if ( ! isInMemory(field) && ! summary.isFromDisk()) { - deployLogger.log(Level.WARNING, summaryField + " in " + summary + " references " + - source + ", which is not an attribute: Using this " + - "summary will cause disk accesses. " + - "Set 'from-disk' on this summary class to silence this warning."); + // TODO: Set to warning on vespa 7 + deployLogger.log(Level.FINE, summaryField + " in " + summary + " references " + + source + ", which is not an attribute: Using this " + + "summary will cause disk accesses. " + + "Set 'from-disk' on this summary class to silence this warning."); } } } diff --git a/config-model/src/test/java/com/yahoo/searchdefinition/SummaryTestCase.java b/config-model/src/test/java/com/yahoo/searchdefinition/SummaryTestCase.java index 5495ba2fffe..6fa716d9b76 100644 --- a/config-model/src/test/java/com/yahoo/searchdefinition/SummaryTestCase.java +++ b/config-model/src/test/java/com/yahoo/searchdefinition/SummaryTestCase.java @@ -30,7 +30,7 @@ public class SummaryTestCase { DeployLoggerStub logger = new DeployLoggerStub(); SearchBuilder.createFromFile("src/test/examples/disksummary.sd", logger); assertEquals(1, logger.entries.size()); - assertEquals(Level.WARNING, logger.entries.get(0).level); + assertEquals(Level.FINE, logger.entries.get(0).level); assertEquals("summary field 'ondisk' in document summary 'default' references source field 'ondisk', " + "which is not an attribute: Using this summary will cause disk accesses. " + "Set 'from-disk' on this summary class to silence this warning.", |