[doris] branch master updated: [fix](nereids)fix bug of lost rollup property in create table statement (#25580)

[starocean999]

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 <40539150+starocean...@users.noreply.github.com>
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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org