(doris) branch branch-3.0 updated: branch-3.0: [improve](variant) only sanitize in debug mode #45689 (#45697)

[eldenmoon]

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

eldenmoon 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 68bcdc074d5 branch-3.0: [improve](variant) only sanitize in debug mode 
#45689 (#45697)
68bcdc074d5 is described below

commit 68bcdc074d577afdc2ceaefc0b7e08ab467fd043
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Dec 20 13:50:07 2024 +0800

    branch-3.0: [improve](variant) only sanitize in debug mode #45689 (#45697)
    
    Cherry-picked from #45689
    
    Co-authored-by: lihangyu <lihan...@selectdb.com>
---
 be/src/vec/columns/column_object.cpp | 2 ++
 be/src/vec/columns/column_object.h   | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/be/src/vec/columns/column_object.cpp 
b/be/src/vec/columns/column_object.cpp
index f38b810848a..90e277c09c0 100644
--- a/be/src/vec/columns/column_object.cpp
+++ b/be/src/vec/columns/column_object.cpp
@@ -1946,6 +1946,7 @@ std::string ColumnObject::debug_string() const {
 }
 
 Status ColumnObject::sanitize() const {
+#ifndef NDEBUG
     RETURN_IF_CATCH_EXCEPTION(check_consistency());
     for (const auto& subcolumn : subcolumns) {
         if (subcolumn->data.is_finalized()) {
@@ -1960,6 +1961,7 @@ Status ColumnObject::sanitize() const {
     }
 
     VLOG_DEBUG << "sanitized " << debug_string();
+#endif
     return Status::OK();
 }
 
diff --git a/be/src/vec/columns/column_object.h 
b/be/src/vec/columns/column_object.h
index 564082b0ef4..1ace552d91d 100644
--- a/be/src/vec/columns/column_object.h
+++ b/be/src/vec/columns/column_object.h
@@ -441,7 +441,7 @@ public:
 
     bool empty() const;
 
-    // Check if all columns and types are aligned
+    // Check if all columns and types are aligned, only in debug mode
     Status sanitize() const;
 
     std::string debug_string() const;


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