Skip to content

Commit 74c2eaf

Browse files
committed
Update ASTBuilderTest for chart: default args are handled in rel node visitor
Signed-off-by: Yuanchun Shen <[email protected]>
1 parent 328f987 commit 74c2eaf

File tree

1 file changed

+4
-16
lines changed

1 file changed

+4
-16
lines changed

ppl/src/test/java/org/opensearch/sql/ppl/parser/AstBuilderTest.java

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1249,10 +1249,7 @@ public void testTimechartWithPerSecondFunction() {
12491249
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
12501250
.columnSplit(null)
12511251
.aggregationFunction(alias("per_second(a)", aggregate("sum", field("a"))))
1252-
.arguments(
1253-
exprList(
1254-
argument("limit", intLiteral(10)),
1255-
argument("useother", booleanLiteral(true))))
1252+
.arguments(exprList())
12561253
.build(),
12571254
let(
12581255
field("per_second(a)"),
@@ -1281,10 +1278,7 @@ public void testTimechartWithPerMinuteFunction() {
12811278
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
12821279
.columnSplit(null)
12831280
.aggregationFunction(alias("per_minute(a)", aggregate("sum", field("a"))))
1284-
.arguments(
1285-
exprList(
1286-
argument("limit", intLiteral(10)),
1287-
argument("useother", booleanLiteral(true))))
1281+
.arguments(exprList())
12881282
.build(),
12891283
let(
12901284
field("per_minute(a)"),
@@ -1313,10 +1307,7 @@ public void testTimechartWithPerHourFunction() {
13131307
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
13141308
.columnSplit(null)
13151309
.aggregationFunction(alias("per_hour(a)", aggregate("sum", field("a"))))
1316-
.arguments(
1317-
exprList(
1318-
argument("limit", intLiteral(10)),
1319-
argument("useother", booleanLiteral(true))))
1310+
.arguments(exprList())
13201311
.build(),
13211312
let(
13221313
field("per_hour(a)"),
@@ -1345,10 +1336,7 @@ public void testTimechartWithPerDayFunction() {
13451336
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
13461337
.columnSplit(null)
13471338
.aggregationFunction(alias("per_day(a)", aggregate("sum", field("a"))))
1348-
.arguments(
1349-
exprList(
1350-
argument("limit", intLiteral(10)),
1351-
argument("useother", booleanLiteral(true))))
1339+
.arguments(exprList())
13521340
.build(),
13531341
let(
13541342
field("per_day(a)"),

0 commit comments

Comments
 (0)