This is an automated email from the ASF dual-hosted git repository.
morrysnow pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.1 by this push:
new 71e1a18458a branch-3.1: [Fix](compaction) Remove incorrect DCHECK
#54650 (#54745)
71e1a18458a is described below
commit 71e1a18458a70c5b4fdce2d90631f38ecfaccb95
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Aug 19 10:16:29 2025 +0800
branch-3.1: [Fix](compaction) Remove incorrect DCHECK #54650 (#54745)
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 d683d94afe7..b398454ba33 100644
--- a/be/src/cloud/cloud_tablet.cpp
+++ b/be/src/cloud/cloud_tablet.cpp
@@ -1138,13 +1138,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 73a295a590e..f4930c9f3da 100644
--- a/be/src/olap/compaction.cpp
+++ b/be/src/olap/compaction.cpp
@@ -1217,14 +1217,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]
