From 3f95275f2397e06dcc7e6634e661b6b6e5e1fedf Mon Sep 17 00:00:00 2001 From: Robert Muir Date: Mon, 29 Jun 2015 13:20:53 -0400 Subject: [PATCH] Add back check with a note, so anyone reviewing isn't confused. --- core/src/main/java/org/elasticsearch/bootstrap/Security.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/core/src/main/java/org/elasticsearch/bootstrap/Security.java b/core/src/main/java/org/elasticsearch/bootstrap/Security.java index 496ff7b1b6a..6bd000e37e4 100644 --- a/core/src/main/java/org/elasticsearch/bootstrap/Security.java +++ b/core/src/main/java/org/elasticsearch/bootstrap/Security.java @@ -87,6 +87,11 @@ final class Security { for (Map.Entry e : SPECIAL_JARS.entrySet()) { if (e.getKey().matcher(url.getPath()).matches()) { String prop = e.getValue(); + // TODO: we need to fix plugins to not include duplicate e.g. lucene-core jars, + // to add back this safety check! see https://github.com/elastic/elasticsearch/issues/11647 + // if (System.getProperty(prop) != null) { + // throw new IllegalStateException("property: " + prop + " is unexpectedly set: " + System.getProperty(prop)); + //} System.setProperty(prop, url.toString()); } }