Clean up

Author: julian-elastic

x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/expression/predicate/regex/RLikePatternList.java

@@ -22,7 +22,6 @@ public class RLikePatternList extends AbstractStringPattern implements Writeable
     private final List<RLikePattern> patternList;
 
     public RLikePatternList(List<RLikePattern> patternList) {
-
         this.patternList = patternList;
     }
 
@@ -69,11 +68,9 @@ public boolean equals(Object obj) {
         if (this == obj) {
             return true;
         }
-
         if (obj == null || getClass() != obj.getClass()) {
             return false;
         }
-
         RLikePatternList other = (RLikePatternList) obj;
         return patternList.equals(other.patternList);
     }

x-pack/plugin/esql/qa/server/single-node/src/javaRestTest/java/org/elasticsearch/xpack/esql/qa/single_node/PushQueriesIT.java

@@ -302,7 +302,7 @@ public void testRLikeList() throws IOException {
         String luceneQuery = switch (type) {
             case CONSTANT_KEYWORD, MATCH_ONLY_TEXT_WITH_KEYWORD, AUTO, TEXT_WITH_KEYWORD -> "*:*";
             case SEMANTIC_TEXT_WITH_KEYWORD -> "FieldExistsQuery [field=_primary_term]";
-            case KEYWORD -> "test:AutomatonQuery";
+            case KEYWORD -> "test:RLIKE(\"%value.*\", \"abc*\"), caseInsensitive=false";
         };
         ComputeSignature dataNodeSignature = switch (type) {
             case CONSTANT_KEYWORD, KEYWORD -> ComputeSignature.FILTER_IN_QUERY;

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/regex/RLikeList.java

@@ -34,7 +34,7 @@ public class RLikeList extends RegexMatch<RLikePatternList> {
     );
 
     /**
-     * The documentation for this function is in RLike, and shown to the users `RLIKE` in the docs.
+     * The documentation for this function is in RLike, and shown to the users as `RLIKE` in the docs.
      */
     public RLikeList(
         Source source,
@@ -85,14 +85,12 @@ public Translatable translatable(LucenePushdownPredicates pushdownPredicates) {
         return pushdownPredicates.isPushableAttribute(field()) ? Translatable.YES : Translatable.NO;
     }
 
-
     /**
      * Returns a {@link Query} that matches the field against the provided patterns.
      * For now, we only support a single pattern in the list for pushdown.
      */
     @Override
     public Query asQuery(LucenePushdownPredicates pushdownPredicates, TranslatorHandler handler) {
-        //throw new RuntimeException("As query called");
         var field = field();
         LucenePushdownPredicates.checkIsPushableAttribute(field);
         return translateField(handler.nameOf(field instanceof FieldAttribute fa ? fa.exactAttribute() : field));

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/CsvTests.java

@@ -40,7 +40,6 @@
 import org.elasticsearch.tasks.CancellableTask;
 import org.elasticsearch.tasks.TaskId;
 import org.elasticsearch.test.ESTestCase;
-import org.elasticsearch.test.junit.annotations.TestLogging;
 import org.elasticsearch.threadpool.FixedExecutorBuilder;
 import org.elasticsearch.threadpool.TestThreadPool;
 import org.elasticsearch.threadpool.ThreadPool;