summaryrefslogtreecommitdiff
path: root/core/src/main/java/org/elasticsearch/search/aggregations
diff options
context:
space:
mode:
authorTanguy Leroux <tlrx.dev@gmail.com>2017-05-18 14:57:12 +0200
committerGitHub <noreply@github.com>2017-05-18 14:57:12 +0200
commit96dac4c6704459522d9d2e1700355a4600fa5b94 (patch)
treec82ea6573d34579ca156648ea8eca1431be97f71 /core/src/main/java/org/elasticsearch/search/aggregations
parent779cf3d468caef74402b7bf67cca4bb0eb8318ff (diff)
Remove the unused SignificantTerms.compareTerm() method (#24714)
This method is not used and not tested. While it exists it forces implementations of the interface to implement it while it's unused.
Diffstat (limited to 'core/src/main/java/org/elasticsearch/search/aggregations')
-rw-r--r--core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantLongTerms.java5
-rw-r--r--core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantStringTerms.java6
-rw-r--r--core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantTerms.java2
3 files changed, 0 insertions, 13 deletions
diff --git a/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantLongTerms.java b/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantLongTerms.java
index c88c7c8e1f..29a8e16d9b 100644
--- a/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantLongTerms.java
+++ b/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantLongTerms.java
@@ -77,11 +77,6 @@ public class SignificantLongTerms extends InternalMappedSignificantTerms<Signifi
}
@Override
- public int compareTerm(SignificantTerms.Bucket other) {
- return Long.compare(term, ((Number) other.getKey()).longValue());
- }
-
- @Override
public String getKeyAsString() {
return format.format(term);
}
diff --git a/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantStringTerms.java b/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantStringTerms.java
index 6627a2295d..b170171f5c 100644
--- a/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantStringTerms.java
+++ b/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantStringTerms.java
@@ -28,7 +28,6 @@ import org.elasticsearch.search.aggregations.bucket.significant.heuristics.Signi
import org.elasticsearch.search.aggregations.pipeline.PipelineAggregator;
import java.io.IOException;
-import java.util.Iterator;
import java.util.List;
import java.util.Map;
import java.util.Objects;
@@ -83,11 +82,6 @@ public class SignificantStringTerms extends InternalMappedSignificantTerms<Signi
}
@Override
- public int compareTerm(SignificantTerms.Bucket other) {
- return termBytes.compareTo(((Bucket) other).termBytes);
- }
-
- @Override
public String getKeyAsString() {
return format.format(termBytes);
}
diff --git a/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantTerms.java b/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantTerms.java
index ae09feef93..8c0da8b890 100644
--- a/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantTerms.java
+++ b/core/src/main/java/org/elasticsearch/search/aggregations/bucket/significant/SignificantTerms.java
@@ -40,8 +40,6 @@ public interface SignificantTerms extends MultiBucketsAggregation, Iterable<Sign
long getSupersetSize();
long getSubsetSize();
-
- int compareTerm(SignificantTerms.Bucket other);
}
@Override