f

Author: yiguolei

be/src/util/runtime_profile.h

@@ -191,21 +191,39 @@ class RuntimeProfile {
         virtual Counter* clone() const { return new Counter(type(), value(), _level); }
 
         virtual void update(int64_t delta) {
+#ifndef NDEBUG
+            int64_t prev_value = _value.load(std::memory_order_seq_cst);
+#endif
             _value.fetch_add(delta, std::memory_order_relaxed);
-            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L);
+#ifndef NDEBUG
+            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L)
+                    << " delta: " << delta << " prev_value: " << prev_value;
+#endif
         }
 
         void bit_or(int64_t delta) { _value.fetch_or(delta, std::memory_order_relaxed); }
 
         virtual void set(int64_t value) {
+#ifndef NDEBUG
+            int64_t prev_value = _value.load(std::memory_order_seq_cst);
+#endif
             _value.store(value, std::memory_order_relaxed);
-            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L);
+#ifndef NDEBUG
+            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L)
+                    << " new value: " << value << " prev_value: " << prev_value;
+#endif
         }
 
         virtual void set(double value) {
             DCHECK_EQ(sizeof(value), sizeof(int64_t));
+#ifndef NDEBUG
+            int64_t prev_value = _value.load(std::memory_order_seq_cst);
+#endif
             _value.store(binary_cast<double, int64_t>(value), std::memory_order_relaxed);
-            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L);
+#ifndef NDEBUG
+            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L)
+                    << " new value: " << value << " prev_value: " << prev_value;
+#endif
         }
 
         virtual int64_t value() const { return _value.load(std::memory_order_relaxed); }
@@ -337,9 +355,18 @@ class RuntimeProfile {
         }
 
         void set(int64_t v) override {
+#ifndef NDEBUG
+            int64_t prev_value = current_value_.load(std::memory_order_seq_cst);
+            int64_t prev_max_value = value_.load(std::memory_order_seq_cst);
+#endif
             current_value_.store(v, std::memory_order_relaxed);
             UpdateMax(v);
-            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L);
+#ifndef NDEBUG
+            DCHECK_GT(current_value_.load(std::memory_order_seq_cst), -1L)
+                    << " prev_value: " << prev_value;
+            DCHECK_GT(_value.load(std::memory_order_seq_cst), -1L)
+                    << " prev_max_value: " << prev_max_value << " prev_value: " << prev_value;
+#endif
         }
 
         int64_t current_value() const { return current_value_.load(std::memory_order_relaxed); }