cleanups (elastic#117997)

Author: idegtiarenko

x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/planner/ExpressionTranslators.java

@@ -107,9 +107,7 @@ protected Query asQuery(Not not, TranslatorHandler handler) {
         }
 
         public static Query doTranslate(Not not, TranslatorHandler handler) {
-            Query wrappedQuery = handler.asQuery(not.field());
-            Query q = wrappedQuery.negate(not.source());
-            return q;
+            return handler.asQuery(not.field()).negate(not.source());
         }
     }
 

x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/util/CollectionUtils.java

@@ -30,12 +30,8 @@ public static <T> List<T> combine(List<? extends T> left, List<? extends T> righ
         }
 
         List<T> list = new ArrayList<>(left.size() + right.size());
-        if (left.isEmpty() == false) {
-            list.addAll(left);
-        }
-        if (right.isEmpty() == false) {
-            list.addAll(right);
-        }
+        list.addAll(left);
+        list.addAll(right);
         return list;
     }
 
@@ -73,13 +69,6 @@ public static <T> List<T> combine(Collection<? extends T> left, T... entries) {
         return list;
     }
 
-    public static int mapSize(int size) {
-        if (size < 2) {
-            return size + 1;
-        }
-        return (int) (size / 0.75f + 1f);
-    }
-
     @SafeVarargs
     @SuppressWarnings("varargs")
     public static <T> List<T> nullSafeList(T... entries) {

x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/type/DataTypes.java

@@ -112,41 +112,21 @@ public static DataType fromEs(String name) {
     }
 
     public static DataType fromJava(Object value) {
-        if (value == null) {
-            return NULL;
-        }
-        if (value instanceof Integer) {
-            return INTEGER;
-        }
-        if (value instanceof Long) {
-            return LONG;
-        }
-        if (value instanceof BigInteger) {
-            return UNSIGNED_LONG;
-        }
-        if (value instanceof Boolean) {
-            return BOOLEAN;
-        }
-        if (value instanceof Double) {
-            return DOUBLE;
-        }
-        if (value instanceof Float) {
-            return FLOAT;
-        }
-        if (value instanceof Byte) {
-            return BYTE;
-        }
-        if (value instanceof Short) {
-            return SHORT;
-        }
-        if (value instanceof ZonedDateTime) {
-            return DATETIME;
-        }
-        if (value instanceof String || value instanceof Character) {
-            return KEYWORD;
-        }
-
-        return null;
+        return switch (value) {
+            case null -> NULL;
+            case Integer i -> INTEGER;
+            case Long l -> LONG;
+            case BigInteger bigInteger -> UNSIGNED_LONG;
+            case Boolean b -> BOOLEAN;
+            case Double v -> DOUBLE;
+            case Float v -> FLOAT;
+            case Byte b -> BYTE;
+            case Short s -> SHORT;
+            case ZonedDateTime zonedDateTime -> DATETIME;
+            case String s -> KEYWORD;
+            case Character c -> KEYWORD;
+            default -> null;
+        };
     }
 
     public static boolean isUnsupported(DataType from) {