fixed

apache · Apr 2, 2024 · 02f047f · 02f047f
1 parent dc27c9c
commit 02f047f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/be/src/vec/columns/column_array.h b/be/src/vec/columns/column_array.h
@@ -220,7 +220,7 @@ class ColumnArray final : public COWHelper<IColumn, ColumnArray> {
                              const uint32_t* indices_end) override;
 
     void replace_column_data(const IColumn& rhs, size_t row, size_t self_row = 0) override {
-        DCHECK(self_row != 0 && size() == self_row + 1);
+        DCHECK(self_row == 0 || (self_row != 0 && size() == self_row + 1));
         const auto& r = assert_cast<const ColumnArray&>(rhs);
         const size_t nested_row_size = r.size_at(row);
         const size_t r_nested_start_off = r.offset_at(row);

diff --git a/be/src/vec/columns/column_map.h b/be/src/vec/columns/column_map.h
@@ -137,7 +137,7 @@ class ColumnMap final : public COWHelper<IColumn, ColumnMap> {
     }
 
     void replace_column_data(const IColumn& rhs, size_t row, size_t self_row = 0) override {
-        DCHECK(self_row != 0 && size() == self_row + 1);
+        DCHECK(self_row == 0 || (self_row != 0 && size() == self_row + 1));
         const auto& r = assert_cast<const ColumnMap&>(rhs);
         const size_t nested_row_size = r.size_at(row);
         const size_t r_key_nested_start_off = r.offset_at(row);

diff --git a/be/src/vec/columns/column_string.h b/be/src/vec/columns/column_string.h
@@ -552,7 +552,7 @@ class ColumnString final : public COWHelper<IColumn, ColumnString> {
         // replace_column_data() with a batch column data.
         // and this column is cleared at the every beginning. next we replace column one by one, so
         // the self_row is only equals size() - 1.
-        DCHECK(self_row != 0 && size() == self_row + 1);
+        DCHECK(self_row == 0 || (self_row != 0 && size() == self_row + 1));
         const auto& r = assert_cast<const ColumnString&>(rhs);
         auto data = r.get_data_at(row);