database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[fix](nereids)fix bug of lost rollup property in create table statement (#25580)

Browse Source
This commit is contained in:
starocean999
2023-10-20 14:55:21 +08:00
committed by GitHub
parent 26f8c7e352
commit 2cd22e5187
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
View File

@ -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());
}