This is an automated email from the ASF dual-hosted git repository.
starocean999 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 2cd22e51875 [fix](nereids)fix bug of lost rollup property in create
table statement (#25580)
2cd22e51875 is described below
commit 2cd22e518751001b262e922c407a04f35e24ece4
Author: starocean999 <[email protected]>
AuthorDate: Fri Oct 20 14:55:21 2023 +0800
[fix](nereids)fix bug of lost rollup property in create table statement
(#25580)
---
.../java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
b/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
index aa711a8e271..734a215888b 100644
---
a/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
+++
b/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
@@ -1918,7 +1918,7 @@ public class LogicalPlanBuilder extends
DorisParserBaseVisitor<Object> {
ctx.partitionKeys != null ?
visitIdentifierList(ctx.partitionKeys) : null,
ctx.partitions != null ?
visitPartitionsDef(ctx.partitions) : null,
desc,
- ImmutableList.of(),
+ ctx.rollupDefs() != null ?
visitRollupDefs(ctx.rollupDefs()) : ImmutableList.of(),
properties));
} else if (ctx.AS() != null) {
return new
CreateTableCommand(Optional.of(visitQuery(ctx.query())), new CreateTableInfo(
@@ -1934,7 +1934,7 @@ public class LogicalPlanBuilder extends
DorisParserBaseVisitor<Object> {
ctx.partitionKeys != null ?
visitIdentifierList(ctx.partitionKeys) : null,
ctx.partitions != null ?
visitPartitionsDef(ctx.partitions) : null,
desc,
- ImmutableList.of(),
+ ctx.rollupDefs() != null ?
visitRollupDefs(ctx.rollupDefs()) : ImmutableList.of(),
properties));
} else {
throw new AnalysisException("Should contain at least one column in
a table");
@@ -2067,7 +2067,7 @@ public class LogicalPlanBuilder extends
DorisParserBaseVisitor<Object> {
}
@Override
- public Object visitRollupDefs(RollupDefsContext ctx) {
+ public List<RollupDefinition> visitRollupDefs(RollupDefsContext ctx) {
return
ctx.rollups.stream().map(this::visitRollupDef).collect(Collectors.toList());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
