From 72ffe30cfd5aeac44f3ac67b65d8af84b286379e Mon Sep 17 00:00:00 2001 From: Harald Musum Date: Thu, 24 Aug 2023 11:18:01 +0200 Subject: Generate warning about 'file' config type in another place Seeing warnings in config server logs (used when reading application package and creating config) so move to a place that will only be logged when generating config classes --- configgen/src/main/java/com/yahoo/config/codegen/DefLine.java | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'configgen/src/main/java/com/yahoo/config/codegen/DefLine.java') diff --git a/configgen/src/main/java/com/yahoo/config/codegen/DefLine.java b/configgen/src/main/java/com/yahoo/config/codegen/DefLine.java index 48e1b58816e..fc54f03ad8b 100644 --- a/configgen/src/main/java/com/yahoo/config/codegen/DefLine.java +++ b/configgen/src/main/java/com/yahoo/config/codegen/DefLine.java @@ -34,6 +34,10 @@ public class DefLine { StringBuilder sb = new StringBuilder(line); int parsed = parseNameType(sb); sb.delete(0, parsed); + if (type.name.equals("file")) { + // Note: 'file' is used internally and also there is no support for 'path' in C++, so cannot be removed yet + System.out.println("Warning: config type 'file' is deprecated, use 'path' instead"); + } if (type.name.equals("enum")) { parsed = parseEnum(sb); sb.delete(0, parsed); -- cgit v1.2.3