This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new 24c142ab96 Revert "[fix](merge-on-write) add check for segment num 
(#14032)"
24c142ab96 is described below

commit 24c142ab96695c89df7dc339c0d672c0d6e98ff3
Author: morningman <morning...@163.com>
AuthorDate: Fri Feb 24 09:32:52 2023 +0800

    Revert "[fix](merge-on-write) add check for segment num (#14032)"
    
    This reverts commit 5002f5adcb54a3907d384f98d7e2eee5e9e83276.
    cherry-pick #17058
---
 be/src/olap/rowset/beta_rowset_writer.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/olap/rowset/beta_rowset_writer.cpp 
b/be/src/olap/rowset/beta_rowset_writer.cpp
index 100d85e452..a18d8c5f67 100644
--- a/be/src/olap/rowset/beta_rowset_writer.cpp
+++ b/be/src/olap/rowset/beta_rowset_writer.cpp
@@ -849,7 +849,7 @@ void 
BetaRowsetWriter::_build_rowset_meta(std::shared_ptr<RowsetMeta> rowset_met
     }
 
     rowset_meta->set_num_segments(num_seg);
-    CHECK_EQ(num_seg, segments_encoded_key_bounds.size());
+    // TODO(zhangzhengyu): key_bounds.size() should equal num_seg, but 
currently not always
     rowset_meta->set_num_rows(num_rows_written + _num_rows_written);
     rowset_meta->set_total_disk_size(total_data_size + _total_data_size);
     rowset_meta->set_data_disk_size(total_data_size + _total_data_size);


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

Reply via email to