elasticsearch-analysis-annotation icon indicating copy to clipboard operation
elasticsearch-analysis-annotation copied to clipboard

ElasticSearchIllegalArgumentException is now ElasticsearchIllegalArgumentException

Open jconwell opened this issue 10 years ago • 1 comments

I pulled down your source and updated the pom to reference elasticsearch 1.6.0. Looks like along the way they lower cased the S in ElasticsearchIllegalArgumentException.

I can make a patch that fixes this and brings it up to date with latest elasticsearch if you want.

jconwell avatar Jul 09 '15 21:07 jconwell

Change is in 1 file (InlineAnnotationFilter.java) (and pom.xml of course).

diff --git a/pom.xml b/pom.xml
index 2ee0f74..426fb96 100644
--- a/pom.xml
+++ b/pom.xml
@@ -24,8 +24,8 @@
     </parent>

     <properties>
-        <elasticsearch.version>0.90.1</elasticsearch.version>
-        <lucene.version>4.3.0</lucene.version>
+        <elasticsearch.version>1.6.0</elasticsearch.version>
+        <lucene.version>4.10.4</lucene.version>
     </properties>

     <repositories>
diff --git a/src/main/java/org/elasticsearch/plugin/analysis/annotation/InlineAnnotationFilter.java b/src/main/java/org/elasticsearch/plugin/analysis/annotation/InlineAnnotationFilter.java
index 95d85bb..1f93645 100644
--- a/src/main/java/org/elasticsearch/plugin/analysis/annotation/InlineAnnotationFilter.java
+++ b/src/main/java/org/elasticsearch/plugin/analysis/annotation/InlineAnnotationFilter.java
@@ -28,7 +28,7 @@ import org.apache.lucene.analysis.tokenattributes.CharTermAttribute;
 import org.apache.lucene.analysis.tokenattributes.PositionIncrementAttribute;
 import org.apache.lucene.analysis.tokenattributes.TypeAttribute;
 import org.apache.lucene.util.AttributeSource;
-import org.elasticsearch.ElasticSearchIllegalArgumentException;
+import org.elasticsearch.ElasticsearchIllegalArgumentException;
 import org.elasticsearch.common.settings.Settings;

 /**
@@ -181,7 +181,7 @@ public class InlineAnnotationFilter extends TokenFilter {

         if (start_delim != null) {
            if (start_delim.length() == 0) {
-               throw new ElasticSearchIllegalArgumentException(
+               throw new ElasticsearchIllegalArgumentException(
                        "Analyzer " + name + " has invalid settings: start " +
                                "delimiter cannot be empty string");
            }
@@ -190,7 +190,7 @@ public class InlineAnnotationFilter extends TokenFilter {

         if (end_delim != null) {
            if (end_delim.length() == 0) {
-               throw new ElasticSearchIllegalArgumentException(
+               throw new ElasticsearchIllegalArgumentException(
                        "Analyzer " + name + " has invalid settings: end " +
                                "delimiter cannot be empty string");
            }
@@ -207,7 +207,7 @@ public class InlineAnnotationFilter extends TokenFilter {

         if (delimiter != null) {
            if (delimiter.length() == 0) {
-               throw new ElasticSearchIllegalArgumentException(
+               throw new ElasticsearchIllegalArgumentException(
                        "Analyzer " + name + " has invalid settings: " +
                                "delimiter cannot be empty string");
            }

sujitpal avatar Jul 28 '15 15:07 sujitpal