Skip to content

Commit 64b1287

Browse files
authored
Apply statically imported Arrays.asList (#68776)
The upstream master merge applied a few Arrays.asList to onen class, RestSqlTestCase; the method has been statically imported in previous PR in the branch (and class Arrays import removed), since `asList` is used up to four times on a line, in some tests. This adjusts the merged in code.
1 parent 935cbb3 commit 64b1287

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

x-pack/plugin/sql/qa/server/src/main/java/org/elasticsearch/xpack/sql/qa/rest/RestSqlTestCase.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -436,16 +436,16 @@ public void testCountAndCountDistinct() throws IOException {
436436
boolean columnar = randomBoolean();
437437
expected.put(
438438
"columns",
439-
Arrays.asList(
439+
asList(
440440
columnInfo(mode, "gender", "text", JDBCType.VARCHAR, Integer.MAX_VALUE),
441441
columnInfo(mode, "cnt", "long", JDBCType.BIGINT, 20),
442442
columnInfo(mode, "cnt_dist", "long", JDBCType.BIGINT, 20)
443443
)
444444
);
445445
if (columnar) {
446-
expected.put("values", Arrays.asList(Arrays.asList("f", "m"), Arrays.asList(6, 5), Arrays.asList(3, 3)));
446+
expected.put("values", asList(asList("f", "m"), asList(6, 5), asList(3, 3)));
447447
} else {
448-
expected.put("rows", Arrays.asList(Arrays.asList("f", 6, 3), Arrays.asList("m", 5, 3)));
448+
expected.put("rows", asList(asList("f", 6, 3), asList("m", 5, 3)));
449449
}
450450

451451
Map<String, Object> response = runSql(

0 commit comments

Comments
 (0)