Replace Collections.singletonList with List.of

Signed-off-by: currantw <[email protected]>
opensearch-project · Dec 9, 2024 · 74bbe73 · 74bbe73
1 parent 823cffe
commit 74bbe73
Show file tree

Hide file tree

Showing 99 changed files with 285 additions and 426 deletions.
diff --git a/async-query/src/main/java/org/opensearch/sql/spark/cluster/ClusterManagerEventListener.java b/async-query/src/main/java/org/opensearch/sql/spark/cluster/ClusterManagerEventListener.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.sql.spark.cluster;
 
-import static java.util.Collections.singletonList;
 import static org.opensearch.sql.spark.data.constants.SparkConstants.SPARK_REQUEST_BUFFER_INDEX_NAME;
 
 import com.google.common.annotations.VisibleForTesting;
@@ -183,7 +182,7 @@ private void cancel(Cancellable cron) {
 
   @VisibleForTesting
   public List<Cancellable> getFlintIndexRetentionCron() {
-    return singletonList(flintIndexRetentionCron);
+    return List.of(flintIndexRetentionCron);
   }
 
   private String executorName() {

diff --git a/.../test/java/org/opensearch/sql/common/interceptors/BasicAuthenticationInterceptorTest.java b/.../test/java/org/opensearch/sql/common/interceptors/BasicAuthenticationInterceptorTest.java
@@ -7,8 +7,7 @@
 
 package org.opensearch.sql.common.interceptors;
 
-import static java.util.Collections.singletonList;
-
+import java.util.List;
 import lombok.SneakyThrows;
 import okhttp3.Credentials;
 import okhttp3.Interceptor;
@@ -48,7 +47,6 @@ void testIntercept() {
     Mockito.verify(chain).proceed(requestArgumentCaptor.capture());
     Request request = requestArgumentCaptor.getValue();
     Assertions.assertEquals(
-        singletonList(Credentials.basic("testAdmin", "testPassword")),
-        request.headers("Authorization"));
+        List.of(Credentials.basic("testAdmin", "testPassword")), request.headers("Authorization"));
   }
 }
diff --git a/core/src/main/java/org/opensearch/sql/analysis/ExpressionAnalyzer.java b/core/src/main/java/org/opensearch/sql/analysis/ExpressionAnalyzer.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.sql.analysis;
 
-import static java.util.Collections.singletonList;
 import static org.opensearch.sql.ast.dsl.AstDSL.and;
 import static org.opensearch.sql.ast.dsl.AstDSL.compare;
 
@@ -83,7 +82,7 @@ public Expression visitCast(Cast node, AnalysisContext context) {
     final Expression expression = node.getExpression().accept(this, context);
     return (Expression)
         repository.compile(
-            context.getFunctionProperties(), node.convertFunctionName(), singletonList(expression));
+            context.getFunctionProperties(), node.convertFunctionName(), List.of(expression));
   }
 
   public ExpressionAnalyzer(BuiltinFunctionRepository repository) {

diff --git a/core/src/main/java/org/opensearch/sql/analysis/SelectExpressionAnalyzer.java b/core/src/main/java/org/opensearch/sql/analysis/SelectExpressionAnalyzer.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.analysis;
 
-import static java.util.Collections.singletonList;
-
 import com.google.common.collect.ImmutableList;
 import java.util.List;
 import java.util.Map;
@@ -55,7 +53,7 @@ public List<NamedExpression> analyze(
 
   @Override
   public List<NamedExpression> visitField(Field node, AnalysisContext context) {
-    return singletonList(DSL.named(node.accept(expressionAnalyzer, context)));
+    return List.of(DSL.named(node.accept(expressionAnalyzer, context)));
   }
 
   @Override
@@ -66,8 +64,7 @@ public List<NamedExpression> visitAlias(Alias node, AnalysisContext context) {
     }
 
     Expression expr = referenceIfSymbolDefined(node, context);
-    return singletonList(
-        DSL.named(unqualifiedNameIfFieldOnly(node, context), expr, node.getAlias()));
+    return List.of(DSL.named(unqualifiedNameIfFieldOnly(node, context), expr, node.getAlias()));
   }
 
   /**

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/AggregateFunction.java b/core/src/main/java/org/opensearch/sql/ast/expression/AggregateFunction.java
@@ -6,7 +6,6 @@
 package org.opensearch.sql.ast.expression;
 
 import static java.util.Collections.emptyList;
-import static java.util.Collections.singletonList;
 
 import java.util.List;
 import lombok.EqualsAndHashCode;
@@ -63,7 +62,7 @@ public AggregateFunction(String funcName, UnresolvedExpression field, Boolean di
 
   @Override
   public List<UnresolvedExpression> getChild() {
-    return singletonList(field);
+    return List.of(field);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/Argument.java b/core/src/main/java/org/opensearch/sql/ast/expression/Argument.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
-
 import java.util.List;
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
@@ -26,7 +24,7 @@ public class Argument extends UnresolvedExpression {
   //    private final DataType valueType;
   @Override
   public List<UnresolvedExpression> getChild() {
-    return singletonList(value);
+    return List.of(value);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/Cast.java b/core/src/main/java/org/opensearch/sql/ast/expression/Cast.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
 import static org.opensearch.sql.expression.function.BuiltinFunctionName.CAST_TO_BOOLEAN;
 import static org.opensearch.sql.expression.function.BuiltinFunctionName.CAST_TO_BYTE;
 import static org.opensearch.sql.expression.function.BuiltinFunctionName.CAST_TO_DATE;
@@ -99,7 +98,7 @@ public FunctionName convertFunctionName() {
 
   @Override
   public List<? extends Node> getChild() {
-    return singletonList(expression);
+    return List.of(expression);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/In.java b/core/src/main/java/org/opensearch/sql/ast/expression/In.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
-
 import java.util.List;
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
@@ -29,7 +27,7 @@ public class In extends UnresolvedExpression {
 
   @Override
   public List<UnresolvedExpression> getChild() {
-    return singletonList(field);
+    return List.of(field);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/Interval.java b/core/src/main/java/org/opensearch/sql/ast/expression/Interval.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
-
 import java.util.List;
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
@@ -30,7 +28,7 @@ public Interval(UnresolvedExpression value, String unit) {
 
   @Override
   public List<UnresolvedExpression> getChild() {
-    return singletonList(value);
+    return List.of(value);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/Not.java b/core/src/main/java/org/opensearch/sql/ast/expression/Not.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
-
 import java.util.List;
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
@@ -24,7 +22,7 @@ public class Not extends UnresolvedExpression {
 
   @Override
   public List<UnresolvedExpression> getChild() {
-    return singletonList(expression);
+    return List.of(expression);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/QualifiedName.java b/core/src/main/java/org/opensearch/sql/ast/expression/QualifiedName.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
 import static java.util.Objects.requireNonNull;
 import static java.util.stream.Collectors.toList;
 
@@ -25,7 +24,7 @@ public class QualifiedName extends UnresolvedExpression {
   private final List<String> parts;
 
   public QualifiedName(String name) {
-    this.parts = singletonList(name);
+    this.parts = List.of(name);
   }
 
   /** QualifiedName Constructor. */

diff --git a/core/src/main/java/org/opensearch/sql/ast/expression/UnresolvedArgument.java b/core/src/main/java/org/opensearch/sql/ast/expression/UnresolvedArgument.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.expression;
 
-import static java.util.Collections.singletonList;
-
 import java.util.List;
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
@@ -28,7 +26,7 @@ public UnresolvedArgument(String argName, UnresolvedExpression value) {
 
   @Override
   public List<UnresolvedExpression> getChild() {
-    return singletonList(value);
+    return List.of(value);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/tree/RareTopN.java b/core/src/main/java/org/opensearch/sql/ast/tree/RareTopN.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.tree;
 
-import static java.util.Collections.singletonList;
-
 import java.util.List;
 import lombok.AllArgsConstructor;
 import lombok.EqualsAndHashCode;
@@ -42,7 +40,7 @@ public RareTopN attach(UnresolvedPlan child) {
 
   @Override
   public List<UnresolvedPlan> getChild() {
-    return singletonList(this.child);
+    return List.of(this.child);
   }
 
   @Override

diff --git a/core/src/main/java/org/opensearch/sql/ast/tree/Relation.java b/core/src/main/java/org/opensearch/sql/ast/tree/Relation.java
@@ -5,8 +5,6 @@
 
 package org.opensearch.sql.ast.tree;
 
-import static java.util.Collections.singletonList;
-
 import com.google.common.collect.ImmutableList;
 import java.util.List;
 import java.util.stream.Collectors;
@@ -33,7 +31,7 @@ public Relation(UnresolvedExpression tableName) {
   }
 
   public Relation(UnresolvedExpression tableName, String alias) {
-    this.tableName = singletonList(tableName);
+    this.tableName = List.of(tableName);
     this.alias = alias;
   }