Merge pull request #30971 from vespa-engine/toregge/log-warning-when-type-is-specified-for-document-summary

Log warning when type is specified for document summary.

3 files changed, 4 insertions, 4 deletions

diff --git a/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedFields.java b/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedFields.java
index 053a5ac777b..7659a1e6562 100644
--- a/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedFields.java
+++ b/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedFields.java
@@ -159,7 +159,7 @@ public class ConvertParsedFields {
             var dataType = field.getDataType();
             var otherType = summaryField.getType();
             if (otherType != null && summaryField.getHasExplicitType()) {
-                schema.getDeployLogger().log(Level.FINE, () -> "For schema '" + schema.getName() +
+                schema.getDeployLogger().log(Level.WARNING, () -> "For schema '" + schema.getName() +
                         "', field '" + field.getName() +
                         "', summary '" + summaryField.name() +
                         "': Specifying the type is deprecated, ignored and will be an error in Vespa 9." +
diff --git a/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedSchemas.java b/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedSchemas.java
index ee15b95b198..3c87044850f 100644
--- a/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedSchemas.java
+++ b/config-model/src/main/java/com/yahoo/schema/parser/ConvertParsedSchemas.java
@@ -233,7 +233,7 @@ public class ConvertParsedSchemas {
             var parsedType = parsedField.getType();
             if (parsedType != null) {
                 var log = schema.getDeployLogger();
-                log.log(Level.FINE, () -> "For schema '" + schema.getName() +
+                log.log(Level.WARNING, () -> "For schema '" + schema.getName() +
                         "', document-summary '" + parsed.name() +
                         "', summary field '" + parsedField.name() +
                         "': Specifying the type is deprecated, ignored and will be an error in Vespa 9." +
diff --git a/config-model/src/test/java/com/yahoo/schema/SummaryTestCase.java b/config-model/src/test/java/com/yahoo/schema/SummaryTestCase.java
index 6442edd547d..8ffbab84fd7 100644
--- a/config-model/src/test/java/com/yahoo/schema/SummaryTestCase.java
+++ b/config-model/src/test/java/com/yahoo/schema/SummaryTestCase.java
@@ -359,7 +359,7 @@ public class SummaryTestCase {
         ApplicationBuilder.createFromStrings(logger, sd);
         if (explicit) {
             assertEquals(1, logger.entries.size());
-            assertEquals(Level.FINE, logger.entries.get(0).level);
+            assertEquals(Level.WARNING, logger.entries.get(0).level);
             assertEquals("For schema 'test', field 'foo', summary 'bar':" +
                     " Specifying the type is deprecated, ignored and will be an error in Vespa 9." +
                     " Remove the type specification to silence this warning.", logger.entries.get(0).message);
@@ -392,7 +392,7 @@ public class SummaryTestCase {
         ApplicationBuilder.createFromStrings(logger, sd);
         if (explicit) {
             assertEquals(1, logger.entries.size());
-            assertEquals(Level.FINE, logger.entries.get(0).level);
+            assertEquals(Level.WARNING, logger.entries.get(0).level);
             assertEquals("For schema 'test', document-summary 'bar', summary field 'foo':" +
                     " Specifying the type is deprecated, ignored and will be an error in Vespa 9." +
                     " Remove the type specification to silence this warning.", logger.entries.get(0).message);