diff options
author | Lester Solbakken <lesters@oath.com> | 2021-01-25 19:41:42 +0100 |
---|---|---|
committer | Lester Solbakken <lesters@oath.com> | 2021-01-25 19:41:42 +0100 |
commit | bbb42508a50350c3099819276b33ec26f1323dd8 (patch) | |
tree | f48a8c85d694dfd9dd6511083fde6186313912e6 /config-model | |
parent | e4fdb0a917b1cf6e06e06e97ff1c2dece61acb09 (diff) |
Log skip of model import to deploy logger
Diffstat (limited to 'config-model')
-rw-r--r-- | config-model/src/main/java/com/yahoo/config/model/deploy/DeployState.java | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/config/model/deploy/DeployState.java b/config-model/src/main/java/com/yahoo/config/model/deploy/DeployState.java index 13222e323f7..046eef65eb5 100644 --- a/config-model/src/main/java/com/yahoo/config/model/deploy/DeployState.java +++ b/config-model/src/main/java/com/yahoo/config/model/deploy/DeployState.java @@ -41,6 +41,7 @@ import com.yahoo.vespa.model.container.search.SemanticRuleBuilder; import com.yahoo.vespa.model.container.search.SemanticRules; import com.yahoo.vespa.model.search.NamedSchema; +import java.io.File; import java.io.FileNotFoundException; import java.io.IOException; import java.io.Reader; @@ -54,6 +55,7 @@ import java.util.Map; import java.util.Objects; import java.util.Optional; import java.util.Set; +import java.util.logging.Level; /** * Contains various state during deploy that should be available in all builders of a {@link com.yahoo.config.model.ConfigModel} @@ -138,8 +140,7 @@ public class DeployState implements ConfigDefinitionStore { this.zone = zone; this.queryProfiles = queryProfiles; // TODO: Remove this by seeing how pagetemplates are propagated this.semanticRules = semanticRules; // TODO: Remove this by seeing how pagetemplates are propagated - this.importedModels = new ImportedMlModels(applicationPackage.getFileReference(ApplicationPackage.MODELS_DIR), - modelImporters); + this.importedModels = importMlModels(applicationPackage, modelImporters, deployLogger); ValidationOverrides suppliedValidationOverrides = applicationPackage.getValidationOverrides().map(ValidationOverrides::fromXml) .orElse(ValidationOverrides.empty); @@ -206,6 +207,18 @@ public class DeployState implements ConfigDefinitionStore { return keyToRepo; } + private static ImportedMlModels importMlModels(ApplicationPackage applicationPackage, + Collection<MlModelImporter> modelImporters, + DeployLogger deployLogger) { + File importFrom = applicationPackage.getFileReference(ApplicationPackage.MODELS_DIR); + ImportedMlModels importedModels = new ImportedMlModels(importFrom, modelImporters); + for (var entry : importedModels.getSkippedModels().entrySet()) { + deployLogger.log(Level.WARNING, "Skipping import of model " + entry.getKey() + " as an exception " + + "occurred during import. Error: " + entry.getValue()); + } + return importedModels; + } + // Global registry of rank profiles. // TODO: I think this can be removed when we remove "<search version=2.0>" and only support content. private final RankProfileRegistry rankProfileRegistry; |