diff options
author | Harald Musum <musum@verizonmedia.com> | 2020-11-09 09:17:53 +0100 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2020-11-09 09:17:53 +0100 |
commit | eb74d778b8610d1ee52d85cd225d61b2b497854b (patch) | |
tree | a9f63333259b4c3b2f663184b8bb31378af350b8 /config-application-package | |
parent | 434567d25d1ccecbabf193ce1692be718ca11ea0 (diff) |
Bundled search dedinition support has been removed
Diffstat (limited to 'config-application-package')
-rw-r--r-- | config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java | 27 |
1 files changed, 4 insertions, 23 deletions
diff --git a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java index 1bdedc503bf..abcf98c38a0 100644 --- a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java +++ b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java @@ -78,7 +78,7 @@ public class FilesApplicationPackage implements ApplicationPackage { * The name of the subdirectory (below the original application package root) * where a preprocessed version of this application package is stored. * As it happens, the config model is first created with an application package in this subdirectory, - * and later used backed by an application package which is not in this subdirectory. + * and later backed by an application package which is not in this subdirectory. * To enable model code to correct for this, this constant must be publicly known. * * All of this stuff is Very Unfortunate and should be fixed. -Jon @@ -304,34 +304,15 @@ public class FilesApplicationPackage implements ApplicationPackage { @Override public Collection<NamedReader> searchDefinitionContents() { - Map<String, NamedReader> ret = new LinkedHashMap<>(); - Set<String> fileSds = new LinkedHashSet<>(); - Set<String> bundleSds = new LinkedHashSet<>(); + Set<NamedReader> ret = new LinkedHashSet<>(); try { for (File f : getSearchDefinitionFiles()) { - fileSds.add(f.getName()); - ret.put(f.getName(), new NamedReader(f.getName(), new FileReader(f))); + ret.add(new NamedReader(f.getName(), new FileReader(f))); } } catch (Exception e) { throw new IllegalArgumentException("Couldn't get search definition contents.", e); } - verifySdsDisjoint(fileSds, bundleSds); - return ret.values(); - } - - /** - * Verify that two sets of search definitions are disjoint (TODO: everything except error message is very generic). - * - * @param fileSds Set of search definitions from file - * @param bundleSds Set of search definitions from bundles - */ - private void verifySdsDisjoint(Set<String> fileSds, Set<String> bundleSds) { - if (!Collections.disjoint(fileSds, bundleSds)) { - Collection<String> disjoint = new ArrayList<>(fileSds); - disjoint.retainAll(bundleSds); - throw new IllegalArgumentException("For the following search definitions names there are collisions between those specified inside " + - "docproc bundles and those in searchdefinitions/ in application package: "+disjoint); - } + return ret; } /** |