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

branch-2.1: [fix](nereids) fix create/alter view column name force cast to lower case #48569 (#48601)

Browse Source 
cherry pick from #48569
This commit is contained in:
yujun
2025-03-05 10:04:25 +08:00
committed by GitHub
parent 4ac431b8c8
commit d81e2ebcc8
5 changed files with 52 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@ -726,8 +726,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);
}
/**

10
regression-test/data/mtmv_p0/test_base_mtmv.out
View File

@ -44,3 +44,13 @@ multi_mv_test_base_mtmv SCHEMA_CHANGE SUCCESS
-- !success --
multi_mv_test_base_mtmv NORMAL SUCCESS
-- !desc_mv_1 --
event_day date Yes true \N
id bigint Yes true \N
username varchar(20) Yes true \N
-- !desc_mv_2 --
event_Day date Yes true \N
Id bigint Yes true \N
UserName varchar(20) Yes true \N

9
regression-test/data/view_p0/view_p0.out
View File

@ -24,3 +24,12 @@ internal regression_test_view_p0 test_view select 1,to_base64(AES_ENCRYPT('doris
-- !sql2 --
-- !desc_view_1 --
c_date varchar(50) Yes false \N
-- !desc_view_2 --
C_DatE varchar(50) Yes false \N
-- !desc_view_3 --
c_dATE varchar(65533) Yes false \N

13
regression-test/suites/mtmv_p0/test_base_mtmv.groovy
View File

@ -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
${querySql};
"""
qt_desc_mv_2 "desc ${mvName}"
sql """drop table if exists `${tableName}`"""
sql """drop table if exists `${newTableName}`"""
sql """ DROP MATERIALIZED VIEW ${mvName}"""

19
regression-test/suites/view_p0/view_p0.groovy
View File

@ -165,6 +165,25 @@ suite("view_p0") {
sql """select * from test_view_table2_view;"""
sql """set enable_nereids_planner=true;"""
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"""
}