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 a1c039ed780 branch-3.0: [fix](case) fix mow cluster key case #43066 
(#44434)
a1c039ed780 is described below

commit a1c039ed78015f0d53f0ea0fc5572c5bba5a122e
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Nov 22 14:34:36 2024 +0800

    branch-3.0: [fix](case) fix mow cluster key case #43066 (#44434)
    
    Cherry-picked from #43066
    
    Co-authored-by: meiyi <myime...@gmail.com>
---
 .../test_new_partial_update_delete.out             |  52 ---------
 .../test_new_partial_update_delete.groovy          | 116 +--------------------
 .../test_partial_update_delete.groovy              |   5 +-
 3 files changed, 5 insertions(+), 168 deletions(-)

diff --git 
a/regression-test/data/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.out
 
b/regression-test/data/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.out
index eb84ec72a67..ebc105e1a25 100644
--- 
a/regression-test/data/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.out
+++ 
b/regression-test/data/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.out
@@ -25,32 +25,6 @@
 -- !sql15 --
 2      3       \N      \N      \N
 
--- !sql21 --
-1      1       1       1       1
-
--- !sql22 --
-
--- !sql23 --
-1      1       1       1       1       1
-
--- !sql24 --
-1      2       \N      \N      \N
-
--- !sql31 --
-1      2       \N      \N      \N
-2      2       2       2       2
-
--- !sql32 --
-
--- !sql33 --
-
--- !sql34 --
-1      2       \N      \N      \N
-2      2       2       2       2
-
--- !sql35 --
-2      3       \N      \N      \N
-
 -- !sql1 --
 1      1       1       1       1
 
@@ -77,29 +51,3 @@
 -- !sql15 --
 2      3       \N      \N      \N
 
--- !sql21 --
-1      1       1       1       1
-
--- !sql22 --
-
--- !sql23 --
-1      1       1       1       1       1
-
--- !sql24 --
-1      2       \N      \N      \N
-
--- !sql31 --
-1      2       \N      \N      \N
-2      2       2       2       2
-
--- !sql32 --
-
--- !sql33 --
-
--- !sql34 --
-1      2       \N      \N      \N
-2      2       2       2       2
-
--- !sql35 --
-2      3       \N      \N      \N
-
diff --git 
a/regression-test/suites/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.groovy
 
b/regression-test/suites/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.groovy
index 99983dde6df..4e91eda1bc9 100644
--- 
a/regression-test/suites/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.groovy
+++ 
b/regression-test/suites/unique_with_mow_c_p0/partial_update/test_new_partial_update_delete.groovy
@@ -65,6 +65,7 @@ suite('test_new_partial_update_delete') {
                     PROPERTIES (
                         "disable_auto_compaction" = "true",
                         "enable_unique_key_merge_on_write" = "false",
+                        "enable_mow_light_delete" = "false",
                         "replication_num" = "1",
                         "store_row_column" = "${use_row_store}"); """
                 sql """alter table ${tableMorName2} set 
("enable_mow_light_delete"="true")"""
@@ -86,6 +87,7 @@ suite('test_new_partial_update_delete') {
                 DISTRIBUTED BY HASH(k1) BUCKETS 1
                 PROPERTIES (
                     "disable_auto_compaction" = "true",
+                    "enable_mow_light_delete" = "false",
                     "replication_num" = "1",
                     "store_row_column" = "${use_row_store}"); """
 
@@ -139,120 +141,6 @@ suite('test_new_partial_update_delete') {
             sql "set enable_insert_strict=true;"
 
             sql "drop table if exists ${tableName1};"
-
-
-
-            // old planner
-            try {
-                def tableMorName3 = "test_new_partial_update_mor_delete3"
-                sql "DROP TABLE IF EXISTS ${tableMorName3};"
-                sql """ CREATE TABLE IF NOT EXISTS ${tableMorName3} (
-                        `k1` int NOT NULL,
-                        `c1` int,
-                        `c2` int,
-                        `c3` int,
-                        `c4` int
-                        )UNIQUE KEY(k1)
-                    DISTRIBUTED BY HASH(k1) BUCKETS 1
-                    PROPERTIES (
-                        "disable_auto_compaction" = "true",
-                        "enable_unique_key_merge_on_write" = "false",
-                        "enable_mow_light_delete" = "true",
-                        "replication_num" = "1",
-                        "store_row_column" = "${use_row_store}"); """
-            } catch (Exception e) {
-                log.info(e.getMessage())
-                assertTrue(e.getMessage().contains('enable_mow_light_delete 
property is only supported for unique merge-on-write table'))
-            }
-
-            try {
-                def tableMorName4 = "test_new_partial_update_mor_delete4"
-                sql "DROP TABLE IF EXISTS ${tableMorName4};"
-                sql """ CREATE TABLE IF NOT EXISTS ${tableMorName4} (
-                        `k1` int NOT NULL,
-                        `c1` int,
-                        `c2` int,
-                        `c3` int,
-                        `c4` int
-                        )UNIQUE KEY(k1)
-                    DISTRIBUTED BY HASH(k1) BUCKETS 1
-                    PROPERTIES (
-                        "disable_auto_compaction" = "true",
-                        "enable_unique_key_merge_on_write" = "false",
-                        "replication_num" = "1",
-                        "store_row_column" = "${use_row_store}"); """
-                sql """alter table ${tableMorName4} set 
("enable_mow_light_delete"="true")"""
-            } catch (Exception e) {
-                log.info(e.getMessage())
-                assertTrue(e.getMessage().contains('enable_mow_light_delete 
property is only supported for unique merge-on-write table'))
-            }
-            sql "set enable_nereids_planner=false"
-            def tableName2 = "test_new_partial_update_delete2"
-            sql "DROP TABLE IF EXISTS ${tableName2};"
-            sql """ CREATE TABLE IF NOT EXISTS ${tableName2} (
-                    `k1` int NOT NULL,
-                    `c1` int,
-                    `c2` int,
-                    `c3` int,
-                    `c4` int
-                    )UNIQUE KEY(k1)
-                CLUSTER BY(c3, c2, c4)    
-                DISTRIBUTED BY HASH(k1) BUCKETS 1
-                PROPERTIES (
-                    "disable_auto_compaction" = "true",
-                    "replication_num" = "1",
-                    "store_row_column" = "${use_row_store}"); """
-
-            def output3 = sql "show create table ${tableName2}"
-            assertTrue output3[0][1].contains("\"enable_mow_light_delete\" = 
\"false\"");
-            sql "insert into ${tableName2} values(1,1,1,1,1)"
-            // 1,1,1,1,1
-            qt_sql21 "select * from ${tableName2} order by k1;"
-            sql "delete from ${tableName2} where k1 = 1"
-            // empty
-            qt_sql22 "select * from ${tableName2} order by k1;"
-            sql "set show_hidden_columns = true;"
-            // 1,null,null,null,1
-            qt_sql23 "select k1,c1,c2,c3,c4,__DORIS_DELETE_SIGN__ from 
${tableName2} order by k1;"
-            sql "set show_hidden_columns = false;"
-            sql "set enable_unique_key_partial_update=true;"
-            sql "set enable_insert_strict=false;"
-            sql "insert into ${tableName2} (k1,c1) values(1,2)"
-            // 1,2,NULL,NULL,NULL
-            qt_sql24 "select * from ${tableName2} order by k1;"
-
-
-
-            sql """alter table ${tableName2} set 
("enable_mow_light_delete"="true") """
-            sql "set enable_unique_key_partial_update=false;"
-            sql "set enable_insert_strict=true;"
-            def output4 = sql "show create table ${tableName2}"
-            assertTrue output4[0][1].contains("\"enable_mow_light_delete\" = 
\"true\"");
-            sql "insert into ${tableName2} values(2,2,2,2,2)"
-            // 1,2,NULL,NULL,NULL
-            // 2,2,2,2,2
-            qt_sql31 "select * from ${tableName2} order by k1;"
-            sql "delete from ${tableName2} where k1 <= 2"
-            // empty
-            qt_sql32 "select * from ${tableName2} order by k1;"
-            sql "set show_hidden_columns = true;"
-            // empty
-            qt_sql33 "select * from ${tableName2} order by k1;"
-            sql "set show_hidden_columns = false;"
-            sql "set skip_delete_predicate = true;"
-            // 1,2,NULL,NULL,NULL
-            // 2,2,2,2,2
-            qt_sql34 "select * from ${tableName2} order by k1;"
-            sql "set skip_delete_predicate = false;"
-            sql "set enable_unique_key_partial_update=true;"
-            sql "set enable_insert_strict=false;"
-            sql "insert into ${tableName2} (k1,c1) values(2,3)"
-            // 2,3,2,2,2
-            qt_sql35 "select * from ${tableName2} order by k1;"
-            sql "set enable_unique_key_partial_update=false;"
-            sql "set enable_insert_strict=true;"
-
-            sql "drop table if exists ${tableName2};"
         }
     }
 
diff --git 
a/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_delete.groovy
 
b/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_delete.groovy
index 67aa6aa2f13..7e623340c48 100644
--- 
a/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_delete.groovy
+++ 
b/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_delete.groovy
@@ -40,6 +40,7 @@ suite('test_partial_update_delete') {
                 PROPERTIES (
                     "enable_unique_key_merge_on_write" = "true",
                     "disable_auto_compaction" = "true",
+                    "enable_mow_light_delete" = "false",
                     "replication_num" = "1",
                     "store_row_column" = "${use_row_store}"); """
 
@@ -52,6 +53,7 @@ suite('test_partial_update_delete') {
                 PROPERTIES (
                     "enable_unique_key_merge_on_write" = "true",
                     "disable_auto_compaction" = "true",
+                    "enable_mow_light_delete" = "false",
                     "replication_num" = "1",
                     "store_row_column" = "${use_row_store}"); """
 
@@ -88,6 +90,7 @@ suite('test_partial_update_delete') {
                 PROPERTIES (
                     "enable_unique_key_merge_on_write" = "true",
                     "disable_auto_compaction" = "true",
+                    "enable_mow_light_delete" = "false",
                     "replication_num" = "1",
                     "store_row_column" = "${use_row_store}"); """
 
@@ -95,13 +98,11 @@ suite('test_partial_update_delete') {
             qt_sql "select k1,c1,c2,c3,c4 from ${tableName3} order by 
k1,c1,c2,c3,c4;"
             streamLoad {
                 table "${tableName3}"
-
                 set 'column_separator', ','
                 set 'format', 'csv'
                 set 'columns', 'k1'
                 set 'partial_columns', 'true'
                 set 'merge_type', 'DELETE'
-
                 file 'partial_update_delete.csv'
                 time 10000
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to