[Fix](nereids)Add semantic check that the hash bucket column must be…

… a key column when creating table for aggregate and unique models
feiniaofeiafei · Mar 8, 2024 · a8d7066 · a8d7066
1 parent eaa74bb
commit a8d7066
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/regression-test/suites/fault_injection_p0/test_rowset_writer_fault.groovy b/regression-test/suites/fault_injection_p0/test_rowset_writer_fault.groovy
@@ -35,7 +35,7 @@ suite("test_rowset_writer_fault", "nonConcurrent") {
             `k13` largeint(40) null comment ""
         ) engine=olap
         UNIQUE KEY (k0)
-        DISTRIBUTED BY HASH(`k1`) BUCKETS 5 properties("replication_num" = "1")
+        DISTRIBUTED BY HASH(`k0`) BUCKETS 5 properties("replication_num" = "1")
         """
 
     GetDebugPoint().clearDebugPointsForAllBEs()

diff --git a/regression-test/suites/insert_p0/test_group_commit_data_bytes_property.groovy b/regression-test/suites/insert_p0/test_group_commit_data_bytes_property.groovy
@@ -51,7 +51,7 @@ suite("test_group_commit_data_bytes_property") {
                 v bigint
                 )  
                 UNIQUE KEY(k)  
-                DISTRIBUTED BY HASH (v) BUCKETS 8
+                DISTRIBUTED BY HASH (k) BUCKETS 8
                 PROPERTIES(  
                 "replication_num" = "1",
                 "group_commit_data_bytes"="1024"

diff --git a/regression-test/suites/insert_p0/test_group_commit_interval_ms_property.groovy b/regression-test/suites/insert_p0/test_group_commit_interval_ms_property.groovy
@@ -51,7 +51,7 @@ suite("test_group_commit_interval_ms_property") {
                 v bigint
                 )  
                 UNIQUE KEY(k)  
-                DISTRIBUTED BY HASH (v) BUCKETS 8
+                DISTRIBUTED BY HASH (k) BUCKETS 8
                 PROPERTIES(  
                 "replication_num" = "1",
                 "group_commit_interval_ms"="10000"

diff --git a/regression-test/suites/schema_change_p0/test_schema_change_agg.groovy b/regression-test/suites/schema_change_p0/test_schema_change_agg.groovy
@@ -176,7 +176,7 @@ suite("test_schema_change_agg", "p0") {
                     partition `old_p1` values [("1"), ("2")),
                     partition `old_p2` values [("2"), ("3"))
                 )
-        DISTRIBUTED BY HASH(pv) BUCKETS 1
+        DISTRIBUTED BY HASH(citycode) BUCKETS 1
         PROPERTIES (
             "replication_num" = "1"
         );
@@ -204,8 +204,8 @@ suite("test_schema_change_agg", "p0") {
     //distribution key
 
     test {
-        sql "alter table ${tableName3} modify column pv bigint sum default '0' comment 'pv'"
-        exception "Can not modify distribution column[pv]. index[${tableName3}]"
+        sql "alter table ${tableName3} modify column citycode smallint  comment 'citycode'"
+        exception "Can not modify distribution column[citycode]. index[${tableName3}]"
     }