This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new 9ec20604558 branch-3.0: [fix](nereids) fix create/alter view column
name force cast to lower case #48569 (#48600)
9ec20604558 is described below
commit 9ec20604558afa2c3e176e90806ba70ba18cbd55
Author: yujun <yu...@selectdb.com>
AuthorDate: Mon Mar 10 12:24:34 2025 +0800
branch-3.0: [fix](nereids) fix create/alter view column name force cast to
lower case #48569 (#48600)
cherry pick from #48569
---
.../doris/nereids/parser/LogicalPlanBuilder.java | 3 +--
regression-test/data/mtmv_p0/test_base_mtmv.out | Bin 987 -> 1193 bytes
regression-test/data/view_p0/view_p0.out | Bin 982 -> 1144 bytes
regression-test/suites/mtmv_p0/test_base_mtmv.groovy | 13 +++++++++++++
regression-test/suites/view_p0/view_p0.groovy | 19 +++++++++++++++++++
5 files changed, 33 insertions(+), 2 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 a77a650a0a3..4e8ebe142b7 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
@@ -751,8 +751,7 @@ public class LogicalPlanBuilder extends
DorisParserBaseVisitor<Object> {
public SimpleColumnDefinition visitSimpleColumnDef(SimpleColumnDefContext
ctx) {
String comment = ctx.STRING_LITERAL() == null ? "" :
LogicalPlanBuilderAssistant.escapeBackSlash(
ctx.STRING_LITERAL().getText().substring(1,
ctx.STRING_LITERAL().getText().length() - 1));
- return new SimpleColumnDefinition(ctx.colName.getText().toLowerCase(),
- comment);
+ return new SimpleColumnDefinition(ctx.colName.getText(), comment);
}
/**
diff --git a/regression-test/data/mtmv_p0/test_base_mtmv.out
b/regression-test/data/mtmv_p0/test_base_mtmv.out
index 6fb0d5f98c2..68073a91bb5 100644
Binary files a/regression-test/data/mtmv_p0/test_base_mtmv.out and
b/regression-test/data/mtmv_p0/test_base_mtmv.out differ
diff --git a/regression-test/data/view_p0/view_p0.out
b/regression-test/data/view_p0/view_p0.out
index 928c01f1aa6..0e59bbd6233 100644
Binary files a/regression-test/data/view_p0/view_p0.out and
b/regression-test/data/view_p0/view_p0.out differ
diff --git a/regression-test/suites/mtmv_p0/test_base_mtmv.groovy
b/regression-test/suites/mtmv_p0/test_base_mtmv.groovy
index ffe221d030b..83880e1c529 100644
--- a/regression-test/suites/mtmv_p0/test_base_mtmv.groovy
+++ b/regression-test/suites/mtmv_p0/test_base_mtmv.groovy
@@ -168,6 +168,19 @@ suite("test_base_mtmv","mtmv") {
order_qt_success "select Name,State,RefreshState from
mv_infos('database'='${dbName}') where Name='${mvName}'"
mv_rewrite_success_without_check_chosen("""${rewriteSql}""", "${mvName}")
+ qt_desc_mv_1 "desc ${mvName}"
+
+ sql """ DROP MATERIALIZED VIEW ${mvName}"""
+ sql """
+ CREATE MATERIALIZED VIEW ${mvName}(event_Day,Id,UserName)
+ BUILD DEFERRED REFRESH COMPLETE ON MANUAL
+ DISTRIBUTED BY RANDOM BUCKETS 2
+ PROPERTIES ('replication_num' = '1')
+ AS
+ SELECT event_day,id,username FROM ${tableName};
+ """
+ qt_desc_mv_2 "desc ${mvName}"
+
sql """drop table if exists `${tableName}`"""
sql """drop table if exists `${newTableName}`"""
sql """ DROP MATERIALIZED VIEW ${mvName}"""
diff --git a/regression-test/suites/view_p0/view_p0.groovy
b/regression-test/suites/view_p0/view_p0.groovy
index f69bd808dc6..e6c6a12f882 100644
--- a/regression-test/suites/view_p0/view_p0.groovy
+++ b/regression-test/suites/view_p0/view_p0.groovy
@@ -172,6 +172,25 @@ suite("view_p0") {
"""
sql """select * from test_view_table2_view;"""
+
+ sql "ALTER VIEW test_view_table2_view as select c_date from
test_view_table2"
+ qt_desc_view_1 "DESC test_view_table2_view"
+
+ sql "ALTER VIEW test_view_table2_view(C_DatE) as select c_date from
test_view_table2"
+ qt_desc_view_2 "DESC test_view_table2_view"
+
+ sql """DROP VIEW IF EXISTS test_view_table2_view"""
+ sql """CREATE VIEW `test_view_table2_view` (c_dATE)
+ AS
+ SELECT
+ date_format(c_date,'%Y-%m-%d') AS `CREATE_DATE`
+ FROM
+ test_view_table2
+ GROUP BY
+ date_format(c_date, '%Y-%m-%d');
+ """
+ qt_desc_view_3 "DESC test_view_table2_view"
+
sql """ drop view if exists test_view_table2_view;"""
sql """DROP TABLE IF EXISTS test_view_table2"""
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
