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 0224143ce84 branch-3.0: [Fix](compaction) Remove incorrect DCHECK
#54650 (#54742)
0224143ce84 is described below
commit 0224143ce847ba1e3733b02d3b8b49a29d5b86a1
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Aug 15 21:03:49 2025 +0800
branch-3.0: [Fix](compaction) Remove incorrect DCHECK #54650 (#54742)
Cherry-picked from #54650
Co-authored-by: bobhan1 <[email protected]>
---
be/src/cloud/cloud_tablet.cpp | 7 -------
be/src/olap/compaction.cpp | 8 --------
2 files changed, 15 deletions(-)
diff --git a/be/src/cloud/cloud_tablet.cpp b/be/src/cloud/cloud_tablet.cpp
index f96f2a9deb7..c2b3e0600df 100644
--- a/be/src/cloud/cloud_tablet.cpp
+++ b/be/src/cloud/cloud_tablet.cpp
@@ -1085,13 +1085,6 @@ Status CloudTablet::calc_delete_bitmap_for_compaction(
RETURN_IF_ERROR(check_rowid_conversion(output_rowset, *location_map));
}
int64_t t5 = MonotonicMicros();
- if (missed_rows) {
- DCHECK_EQ(missed_rows->size(), missed_rows_size);
- if (missed_rows->size() != missed_rows_size) {
- LOG(WARNING) << "missed rows don't match, before: " <<
missed_rows_size
- << " after: " << missed_rows->size();
- }
- }
// 3. store delete bitmap
auto st = _engine.meta_mgr().update_delete_bitmap(*this, -1, initiator,
diff --git a/be/src/olap/compaction.cpp b/be/src/olap/compaction.cpp
index de0dcc79550..2ad9eeed2a2 100644
--- a/be/src/olap/compaction.cpp
+++ b/be/src/olap/compaction.cpp
@@ -1190,14 +1190,6 @@ Status CompactionMixin::modify_rowsets() {
missed_rows.get(), location_map.get(),
*_tablet->tablet_meta()->delete_bitmap(),
&output_rowset_delete_bitmap);
- if (missed_rows) {
- DCHECK_EQ(missed_rows->size(), missed_rows_size);
- if (missed_rows->size() != missed_rows_size) {
- LOG(WARNING) << "missed rows don't match, before: " <<
missed_rows_size
- << " after: " << missed_rows->size();
- }
- }
-
if (location_map) {
RETURN_IF_ERROR(tablet()->check_rowid_conversion(_output_rowset,
*location_map));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
