Use List for tables

Author: willmostly

gateway-ha/src/main/java/io/trino/gateway/ha/router/TrinoQueryProperties.java

@@ -71,7 +71,6 @@
 import java.io.IOException;
 import java.net.URLDecoder;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Enumeration;
 import java.util.List;
 import java.util.Map;
@@ -115,7 +114,7 @@ public TrinoQueryProperties(
             @JsonProperty("body") String body,
             @JsonProperty("queryType") String queryType,
             @JsonProperty("resourceGroupQueryType") String resourceGroupQueryType,
-            @JsonProperty("tables") String[] tables,
+            @JsonProperty("tables") List<String> tables,
             @JsonProperty("defaultCatalog") Optional<String> defaultCatalog,
             @JsonProperty("defaultSchema") Optional<String> defaultSchema,
             @JsonProperty("catalogs") Set<String> catalogs,
@@ -127,7 +126,8 @@ public TrinoQueryProperties(
         this.body = requireNonNullElse(body, "");
         this.queryType = requireNonNullElse(queryType, "");
         this.resourceGroupQueryType = resourceGroupQueryType;
-        this.tables = Arrays.stream(requireNonNullElse(tables, new String[] {})).map(this::parseIdentifierStringToQualifiedName).collect(Collectors.toSet());
+        List<String> defaultTables = ImmutableList.of();
+        this.tables = requireNonNullElse(tables, defaultTables).stream().map(this::parseIdentifierStringToQualifiedName).collect(Collectors.toSet());
         this.defaultCatalog = requireNonNullElse(defaultCatalog, Optional.empty());
         this.defaultSchema = requireNonNullElse(defaultSchema, Optional.empty());
         this.catalogs = requireNonNullElse(catalogs, ImmutableSet.of());

gateway-ha/src/test/java/io/trino/gateway/ha/router/TestTrinoQueryProperties.java

@@ -32,7 +32,7 @@ void testJsonCreator()
                 "SELECT c1 from c.s.t1",
                 "SELECT",
                 "SELECT",
-                new String[] {"c.s.t1"},
+                ImmutableList.of("c.s.t1"),
                 Optional.empty(),
                 Optional.empty(),
                 ImmutableSet.of("c"),
@@ -66,7 +66,7 @@ void testJsonCreatorWithEmptyProperties()
                 "SELECT c1 from c.s.t1",
                 "SELECT",
                 "SELECT",
-                new String[] {},
+                ImmutableList.of(),
                 Optional.empty(),
                 Optional.empty(),
                 ImmutableSet.of(),