Skip to content

Commit 1570c9e

Browse files
committed
Update ASTBuilderTest for chart: default args are handled in rel node visitor
Signed-off-by: Yuanchun Shen <[email protected]>
1 parent 8eeffe4 commit 1570c9e

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
@@ -1234,10 +1234,7 @@ public void testTimechartWithPerSecondFunction() {
12341234
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
12351235
.columnSplit(null)
12361236
.aggregationFunction(alias("per_second(a)", aggregate("sum", field("a"))))
1237-
.arguments(
1238-
exprList(
1239-
argument("limit", intLiteral(10)),
1240-
argument("useother", booleanLiteral(true))))
1237+
.arguments(exprList())
12411238
.build(),
12421239
let(
12431240
field("per_second(a)"),
@@ -1266,10 +1263,7 @@ public void testTimechartWithPerMinuteFunction() {
12661263
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
12671264
.columnSplit(null)
12681265
.aggregationFunction(alias("per_minute(a)", aggregate("sum", field("a"))))
1269-
.arguments(
1270-
exprList(
1271-
argument("limit", intLiteral(10)),
1272-
argument("useother", booleanLiteral(true))))
1266+
.arguments(exprList())
12731267
.build(),
12741268
let(
12751269
field("per_minute(a)"),
@@ -1298,10 +1292,7 @@ public void testTimechartWithPerHourFunction() {
12981292
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
12991293
.columnSplit(null)
13001294
.aggregationFunction(alias("per_hour(a)", aggregate("sum", field("a"))))
1301-
.arguments(
1302-
exprList(
1303-
argument("limit", intLiteral(10)),
1304-
argument("useother", booleanLiteral(true))))
1295+
.arguments(exprList())
13051296
.build(),
13061297
let(
13071298
field("per_hour(a)"),
@@ -1330,10 +1321,7 @@ public void testTimechartWithPerDayFunction() {
13301321
alias("@timestamp", span(field("@timestamp"), intLiteral(1), SpanUnit.of("m"))))
13311322
.columnSplit(null)
13321323
.aggregationFunction(alias("per_day(a)", aggregate("sum", field("a"))))
1333-
.arguments(
1334-
exprList(
1335-
argument("limit", intLiteral(10)),
1336-
argument("useother", booleanLiteral(true))))
1324+
.arguments(exprList())
13371325
.build(),
13381326
let(
13391327
field("per_day(a)"),

0 commit comments

Comments
 (0)