diff options
author | Tanguy Leroux <tlrx.dev@gmail.com> | 2017-05-18 14:57:12 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-05-18 14:57:12 +0200 |
commit | 96dac4c6704459522d9d2e1700355a4600fa5b94 (patch) | |
tree | c82ea6573d34579ca156648ea8eca1431be97f71 /core/src/main/java/org/elasticsearch/search/aggregations | |
parent | 779cf3d468caef74402b7bf67cca4bb0eb8318ff (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')
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 |