fixed small things from PR feedback

not-napoleon · Dec 8, 2023 · 7f19b5a · 7f19b5a
1 parent b870b3f
commit 7f19b5a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/...esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/TestCaseSupplier.java b/...esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/TestCaseSupplier.java
@@ -957,7 +957,6 @@ public TypedData(Object data, String name) {
         public String toString() {
             if (type == DataTypes.UNSIGNED_LONG && data instanceof Long longData) {
                     return type.toString() + "(" + NumericUtils.unsignedLongAsBigInteger(longData).toString() + ")";
-                }
             }
             return type.toString() + "(" + (data == null ? "null" : data.toString()) + ")";
         }

diff --git a/.../org/elasticsearch/xpack/esql/expression/function/scalar/convert/ToUnsignedLongTests.java b/.../org/elasticsearch/xpack/esql/expression/function/scalar/convert/ToUnsignedLongTests.java
@@ -43,7 +43,7 @@ public static Iterable<Object[]> parameters() {
             suppliers,
             read,
             DataTypes.UNSIGNED_LONG,
-            (n) -> n,
+            n -> n,
             BigInteger.ZERO,
             UNSIGNED_LONG_MAX,
             List.of()
@@ -118,7 +118,7 @@ public static Iterable<Object[]> parameters() {
             suppliers,
             evaluatorName.apply("Long"),
             DataTypes.UNSIGNED_LONG,
-            (n) -> BigInteger.valueOf(n),
+            BigInteger::valueOf,
             0L,
             Long.MAX_VALUE,
             List.of()
@@ -142,7 +142,7 @@ public static Iterable<Object[]> parameters() {
             suppliers,
             evaluatorName.apply("Int"),
             DataTypes.UNSIGNED_LONG,
-            (n) -> BigInteger.valueOf(n),
+            BigInteger::valueOf,
             0,
             Integer.MAX_VALUE,
             List.of()