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 53e33c80b24 branch-3.0: [fix](metadata) replace view will break view's
metadata #51058 (#51074)
53e33c80b24 is described below
commit 53e33c80b24b067fba14a0474af12acb76624ee9
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Wed May 21 09:41:12 2025 +0800
branch-3.0: [fix](metadata) replace view will break view's metadata #51058
(#51074)
Cherry-picked from #51058
Co-authored-by: morrySnow <zhangwen...@selectdb.com>
---
.../src/main/java/org/apache/doris/catalog/Env.java | 1 +
.../data/ddl_p0/test_create_or_replace_view.out | Bin 135 -> 505 bytes
.../suites/ddl_p0/test_create_or_replace_view.groovy | 18 ++++++++++++++++++
3 files changed, 19 insertions(+)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
b/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
index 33d16730cf0..e6d0bcd23b9 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
@@ -5704,6 +5704,7 @@ public class Env {
AlterViewStmt alterViewStmt = new
AlterViewStmt(stmt.getTableName(), stmt.getColWithComments(),
stmt.getViewDefStmt());
alterViewStmt.setInlineViewDef(stmt.getInlineViewDef());
+ alterViewStmt.setFinalColumns(stmt.getColumns());
try {
alterView(alterViewStmt);
} catch (UserException e) {
diff --git a/regression-test/data/ddl_p0/test_create_or_replace_view.out
b/regression-test/data/ddl_p0/test_create_or_replace_view.out
index 2448ba68e84..30c254b2386 100644
Binary files a/regression-test/data/ddl_p0/test_create_or_replace_view.out and
b/regression-test/data/ddl_p0/test_create_or_replace_view.out differ
diff --git a/regression-test/suites/ddl_p0/test_create_or_replace_view.groovy
b/regression-test/suites/ddl_p0/test_create_or_replace_view.groovy
index 2ea993580e1..8617d5d0ae1 100644
--- a/regression-test/suites/ddl_p0/test_create_or_replace_view.groovy
+++ b/regression-test/suites/ddl_p0/test_create_or_replace_view.groovy
@@ -31,6 +31,14 @@ suite("test_create_or_replace_view") {
DUPLICATE KEY(k1) DISTRIBUTED BY HASH(k1) BUCKETS 1
PROPERTIES( "replication_num" = "1");
"""
+
+ qt_desc_view_tbl1 """
+ desc test_create_or_replace_view_tbl1
+ """
+ qt_desc_view_tbl2 """
+ desc test_create_or_replace_view_tbl2
+ """
+
sql """INSERT INTO test_create_or_replace_view_tbl1 VALUES(1,1,1)"""
sql """INSERT INTO test_create_or_replace_view_tbl2 VALUES(2,2,2)"""
sql "sync"
@@ -41,6 +49,11 @@ suite("test_create_or_replace_view") {
CREATE VIEW IF NOT EXISTS view_test_create_or_replace_view
AS SELECT * FROM test_create_or_replace_view_tbl1;
"""
+
+ qt_desc_view_upper """
+ desc view_test_create_or_replace_view
+ """
+
qt_sql_1 """select * from view_test_create_or_replace_view"""
sql """
@@ -48,6 +61,11 @@ suite("test_create_or_replace_view") {
AS SELECT * FROM test_create_or_replace_view_tbl2;
"""
qt_sql_2 """select * from view_test_create_or_replace_view"""
+
+ qt_desc_view_upper_after_replace """
+ desc view_test_create_or_replace_view
+ """
+
test {
sql """
CREATE OR REPLACE VIEW IF NOT EXISTS
view_test_create_or_replace_view
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
