Revert "Validate semantic rules when building config model"

This reverts commit fa0044d92067bfe91a104d0a6bb6b085c0b9439e.

2 files changed, 13 insertions, 46 deletions

diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRuleBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRuleBuilder.java
index b184861c102..ce999603439 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRuleBuilder.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRuleBuilder.java
@@ -4,16 +4,11 @@ package com.yahoo.vespa.model.container.search;
 import com.yahoo.config.application.api.ApplicationPackage;
 import com.yahoo.io.IOUtils;
 import com.yahoo.io.reader.NamedReader;
-import com.yahoo.language.simple.SimpleLinguistics;
-import com.yahoo.prelude.semantics.RuleBase;
-import com.yahoo.prelude.semantics.RuleImporter;
-import com.yahoo.prelude.semantics.SemanticRulesConfig;
-import com.yahoo.prelude.semantics.parser.ParseException;
 
 import java.io.File;
 import java.io.IOException;
-import java.util.HashMap;
-import java.util.Map;
+import java.util.List;
+import java.util.stream.Collectors;
 
 /**
  * Reads the semantic rules from the application package by delegating to SemanticRules.
@@ -23,22 +18,16 @@ import java.util.Map;
 // TODO: Move into SemanticRules
 public class SemanticRuleBuilder {
 
-    /** Build the set of semantic rules for an application package and validates them */
+    /** Build the set of semantic rules for an application package */
     public SemanticRules build(ApplicationPackage applicationPackage) {
-        var ruleFiles = applicationPackage.getFiles(ApplicationPackage.RULES_DIR, "sr");
-        var rules = new SemanticRules(ruleFiles.stream().map(this::toRuleBaseConfigView).toList());
-
-        // Create config to make sure rules are valid, config is validated in call to toMap() below
-        var builder = new SemanticRulesConfig.Builder();
-        rules.getConfig(builder);
-        SemanticRulesConfig config = builder.build();
+        List<NamedReader> ruleBaseFiles = null;
         try {
-            toMap(config);  // validates config
-            ensureZeroOrOneDefaultRule(config);
-        } catch (ParseException | IOException e) {
-            throw new RuntimeException(e);
+            ruleBaseFiles = applicationPackage.getFiles(ApplicationPackage.RULES_DIR, "sr");
+            return new SemanticRules(ruleBaseFiles.stream().map(this::toRuleBaseConfigView).toList());
+        }
+        finally {
+            NamedReader.closeAll(ruleBaseFiles);
         }
-        return rules;
     }
 
     private SemanticRules.RuleBase toRuleBaseConfigView(NamedReader reader) {
@@ -54,30 +43,7 @@ public class SemanticRuleBuilder {
 
     private String toName(String fileName) {
         String shortName = new File(fileName).getName();
-        return shortName.substring(0, shortName.length() - ".sr".length());
-    }
-
-    private void ensureZeroOrOneDefaultRule(SemanticRulesConfig config) {
-        String defaultName = null;
-        for (SemanticRulesConfig.Rulebase ruleBase : config.rulebase()) {
-            if (defaultName != null && ruleBase.isdefault())
-                throw new IllegalArgumentException("Both '" + defaultName + "' and '" + ruleBase.name() +
-                                                    "' is marked as default rule, there can only be one");
-            if (ruleBase.isdefault())
-                defaultName = ruleBase.name();
-        }
-    }
-
-    static Map<String, RuleBase> toMap(SemanticRulesConfig config) throws ParseException, IOException {
-        RuleImporter ruleImporter = new RuleImporter(config, new SimpleLinguistics());
-        Map<String, RuleBase> ruleBaseMap = new HashMap<>();
-        for (SemanticRulesConfig.Rulebase ruleBaseConfig : config.rulebase()) {
-            RuleBase ruleBase = ruleImporter.importConfig(ruleBaseConfig);
-            if (ruleBaseConfig.isdefault())
-                ruleBase.setDefault(true);
-            ruleBaseMap.put(ruleBase.getName(), ruleBase);
-        }
-        return ruleBaseMap;
+        return shortName.substring(0, shortName.length()-".sr".length());
     }
 
 }
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRules.java b/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRules.java
index 0083fc6b886..46c8577c6e8 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRules.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/container/search/SemanticRules.java
@@ -6,14 +6,14 @@ import java.io.Serializable;
 import java.util.List;
 
 /**
- * Returns the semantic rules config from a set of rule bases.
+ * Returns the semantic rules config form a set of rule bases.
  * Owned by a container cluster
  *
  * @author bratseth
  */
 public class SemanticRules implements Serializable, SemanticRulesConfig.Producer {
 
-    private final List<RuleBase> ruleBases;
+    private List<RuleBase> ruleBases;
 
     public SemanticRules(List<RuleBase> ruleBases) {
         this.ruleBases = ruleBases;
@@ -46,6 +46,7 @@ public class SemanticRules implements Serializable, SemanticRulesConfig.Producer
             return ruleBaseBuilder;
         }
 
+
     }
 
 }