diff --git a/be/src/cloud/cloud_tablet.cpp b/be/src/cloud/cloud_tablet.cpp index ea86f3b40ff1dc..c27353be5235d0 100644 --- a/be/src/cloud/cloud_tablet.cpp +++ b/be/src/cloud/cloud_tablet.cpp @@ -775,7 +775,8 @@ Status CloudTablet::calc_delete_bitmap_for_compaction( } std::unique_ptr> location_map; - if (config::enable_rowid_conversion_correctness_check) { + if (config::enable_rowid_conversion_correctness_check && + tablet_schema()->cluster_key_idxes().empty()) { location_map = std::make_unique>(); LOG(INFO) << "Location Map inited succ for tablet:" << tablet_id(); } diff --git a/be/src/olap/base_tablet.cpp b/be/src/olap/base_tablet.cpp index a499a27b07f6e2..104dc44ebec8ae 100644 --- a/be/src/olap/base_tablet.cpp +++ b/be/src/olap/base_tablet.cpp @@ -1604,10 +1604,6 @@ Status BaseTablet::check_rowid_conversion( VLOG_DEBUG << "check_rowid_conversion, location_map is empty"; return Status::OK(); } - if (!tablet_schema()->cluster_key_idxes().empty()) { - VLOG_DEBUG << "skip check_rowid_conversion for mow tables with cluster keys"; - return Status::OK(); - } std::vector dst_segments; RETURN_IF_ERROR( diff --git a/be/src/olap/compaction.cpp b/be/src/olap/compaction.cpp index c85ce36a1033f5..98d7787550d197 100644 --- a/be/src/olap/compaction.cpp +++ b/be/src/olap/compaction.cpp @@ -965,7 +965,8 @@ Status CompactionMixin::modify_rowsets() { LOG(INFO) << "RowLocation Set inited succ for tablet:" << _tablet->tablet_id(); } std::unique_ptr> location_map; - if (config::enable_rowid_conversion_correctness_check) { + if (config::enable_rowid_conversion_correctness_check && + tablet()->tablet_schema()->cluster_key_idxes().empty()) { location_map = std::make_unique>(); LOG(INFO) << "Location Map inited succ for tablet:" << _tablet->tablet_id(); } diff --git a/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy b/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy index e08ec60457707f..aca1596f863769 100644 --- a/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy +++ b/regression-test/framework/src/main/groovy/org/apache/doris/regression/Config.groovy @@ -1031,6 +1031,7 @@ class Config { excludeDirectorySet.add("schema_change_p0/unique_ck") List excludeCases = ["test_table_properties", "test_create_table" , "test_default_hll", "test_default_pi", "test_default_bitmap_empty" + , "test_full_compaction", "test_full_compaction_by_table_id" // partial update , "txn_insert", "test_update_schema_change", "test_generated_column_update", "test_nested_type_with_rowstore", "test_partial_update_generated_column", "nereids_partial_update_native_insert_stmt" , "partial_update", "nereids_update_on_current_timestamp", "update_on_current_timestamp", "nereids_delete_mow_partial_update", "delete_mow_partial_update", "test_unique_table_auto_inc"