Revert "Added LonghHash for early detection of existing values"

This reverts commit 9cff744.

Author: ivancea

x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/sort/LongTopNUniqueSort.java

@@ -10,7 +10,6 @@
 import org.elasticsearch.common.util.BigArrays;
 import org.elasticsearch.common.util.BinarySearcher;
 import org.elasticsearch.common.util.LongArray;
-import org.elasticsearch.common.util.LongHash;
 import org.elasticsearch.core.Releasable;
 import org.elasticsearch.core.Releasables;
 import org.elasticsearch.search.sort.BucketedSort;
@@ -28,30 +27,15 @@ public class LongTopNUniqueSort implements Releasable {
 
     private final LongArray values;
     private final LongBinarySearcher searcher;
-    /**
-     * Hash holding the unique seen values.
-     * Adding is an O(1) operation to check if the new value is already in the top, and avoid trying to add it again, which is O(log(n)).
-     */
-    private final LongHash seenUniqueValues;
 
     private int count;
 
     public LongTopNUniqueSort(BigArrays bigArrays, SortOrder order, int limit) {
         this.order = order;
         this.limit = limit;
         this.count = 0;
-
-        boolean success = false;
-        try {
-            this.values = bigArrays.newLongArray(limit, false);
-            this.seenUniqueValues = new LongHash(1, bigArrays);
-            this.searcher = new LongBinarySearcher(values, order);
-            success = true;
-        } finally {
-            if (success == false) {
-                close();
-            }
-        }
+        this.values = bigArrays.newLongArray(limit, false);
+        this.searcher = new LongBinarySearcher(values, order);
     }
 
     public boolean collect(long value) {
@@ -65,10 +49,6 @@ public boolean collect(long value) {
             return false;
         }
 
-        if (seenUniqueValues.add(value) < 0) {
-            return true;
-        }
-
         if (count == 0) {
             values.set(0, value);
             count++;
@@ -179,6 +159,6 @@ private boolean betterThan(long lhs, long rhs) {
 
     @Override
     public final void close() {
-        Releasables.close(values, seenUniqueValues);
+        Releasables.close(values);
     }
 }