From 015b3dd6b8707b302f67e2112dee50a63b376441 Mon Sep 17 00:00:00 2001
From: Harald Musum <musum@verizonmedia.com>
Date: Thu, 26 Nov 2020 23:49:58 +0100
Subject: Revert "Bratseth/apply on restart take 2"

---
 .../com/yahoo/config/codegen/BuilderGenerator.java | 42 +++++++++-------------
 configgen/src/test/resources/allfeatures.reference | 14 ++------
 2 files changed, 18 insertions(+), 38 deletions(-)

(limited to 'configgen')

diff --git a/configgen/src/main/java/com/yahoo/config/codegen/BuilderGenerator.java b/configgen/src/main/java/com/yahoo/config/codegen/BuilderGenerator.java
index 2ebe7ad03bf..b3c4d0c5ff8 100644
--- a/configgen/src/main/java/com/yahoo/config/codegen/BuilderGenerator.java
+++ b/configgen/src/main/java/com/yahoo/config/codegen/BuilderGenerator.java
@@ -44,7 +44,7 @@ public class BuilderGenerator {
     }
 
     private static String getSpecialRootBuilderCode(InnerCNode node) {
-        return (node.getParent() == null) ? "\n" + getRootDeclarations() + "\n" : "";
+        return (node.getParent() == null) ? "\n" + getDispatchCode() + "\n" : "";
     }
 
     private static String getBuildMethod(InnerCNode node) {
@@ -53,36 +53,26 @@ public class BuilderGenerator {
                "}\n";
     }
 
-    private static String getRootDeclarations() {
+    private static String getDispatchCode() {
         // Use full path to @Override, as users are free to define an inner node called
         // 'override'. (summarymap.def does)
         // The generated inner 'Override' class would otherwise be mistaken for the
         // annotation.
-        return  "private boolean _applyOnRestart = false;\n" +
-                "\n" +
-                "@java.lang.Override\n" +
-                "public final boolean dispatchGetConfig(ConfigInstance.Producer producer) {\n" +
-                "  if (producer instanceof Producer) {\n" +
-                "    ((Producer)producer).getConfig(this);\n" +
-                "    return true;\n" +
+        return "@java.lang.Override\n" + //
+                "public final boolean dispatchGetConfig(ConfigInstance.Producer producer) {\n" + //
+                "  if (producer instanceof Producer) {\n" + //
+                "    ((Producer)producer).getConfig(this);\n" + //
+                "    return true;\n" + //
                 "  }\n" + //
-                "  return false;\n" +
-                "}\n" +
-                "\n" +
-                "@java.lang.Override\n" +
-                "public final String getDefMd5() { return CONFIG_DEF_MD5; }\n" +
-                "\n" +
-                "@java.lang.Override\n" +
-                "public final String getDefName() { return CONFIG_DEF_NAME; }\n" +
-                "\n" +
-                "@java.lang.Override\n" +
-                "public final String getDefNamespace() { return CONFIG_DEF_NAMESPACE; }\n" +
-                "\n" +
-                "@java.lang.Override\n" +
-                "public final boolean getApplyOnRestart() { return _applyOnRestart; }\n" +
-                "\n" +
-                "@java.lang.Override\n" +
-                "public final void setApplyOnRestart(boolean applyOnRestart) { _applyOnRestart = applyOnRestart; }";
+                "  return false;\n" + //
+                "}\n" + //
+                "\n" + //
+                "@java.lang.Override\n" + //
+                "public final String getDefMd5()       { return CONFIG_DEF_MD5; }\n" + //
+                "@java.lang.Override\n" + //
+                "public final String getDefName()      { return CONFIG_DEF_NAME; }\n" + //
+                "@java.lang.Override\n" + //
+                "public final String getDefNamespace() { return CONFIG_DEF_NAMESPACE; }";
     }
 
     private static String getUninitializedScalars(InnerCNode node) {
diff --git a/configgen/src/test/resources/allfeatures.reference b/configgen/src/test/resources/allfeatures.reference
index 2008bbf6b52..b84f01f380b 100644
--- a/configgen/src/test/resources/allfeatures.reference
+++ b/configgen/src/test/resources/allfeatures.reference
@@ -601,8 +601,6 @@ public final class AllfeaturesConfig extends ConfigInstance {
       return this;
     }
 
-    private boolean _applyOnRestart = false;
-
     @java.lang.Override
     public final boolean dispatchGetConfig(ConfigInstance.Producer producer) {
       if (producer instanceof Producer) {
@@ -613,20 +611,12 @@ public final class AllfeaturesConfig extends ConfigInstance {
     }
 
     @java.lang.Override
-    public final String getDefMd5() { return CONFIG_DEF_MD5; }
-
+    public final String getDefMd5()       { return CONFIG_DEF_MD5; }
     @java.lang.Override
-    public final String getDefName() { return CONFIG_DEF_NAME; }
-
+    public final String getDefName()      { return CONFIG_DEF_NAME; }
     @java.lang.Override
     public final String getDefNamespace() { return CONFIG_DEF_NAMESPACE; }
 
-    @java.lang.Override
-    public final boolean getApplyOnRestart() { return _applyOnRestart; }
-
-    @java.lang.Override
-    public final void setApplyOnRestart(boolean applyOnRestart) { _applyOnRestart = applyOnRestart; }
-
     public AllfeaturesConfig build() {
       return new AllfeaturesConfig(this);
     }
-- 
cgit v1.2.3