Do not call TypeName for trace in tablet transactions if no trace exi…

…sts (#1867) Fixes #1859
ydb-platform · Feb 12, 2024 · 1dd1ce1 · 1dd1ce1
1 parent e7c0e8c
commit 1dd1ce1
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/ydb/core/tablet_flat/tablet_flat_executor.h b/ydb/core/tablet_flat/tablet_flat_executor.h
@@ -307,13 +307,17 @@ class ITransaction : TNonCopyable {
         out << TypeName(*this);
     }
 
-    virtual void SetupTxSpanName() noexcept {
-        TxSpan.Attribute("Type", TypeName(*this));
+    void SetupTxSpanName() noexcept {
+        if (TxSpan) {
+            TxSpan.Attribute("Type", TypeName(*this));
+        }
     }
 
     void SetupTxSpan(NWilson::TTraceId traceId) noexcept {
         TxSpan = NWilson::TSpan(TWilsonTablet::Tablet, std::move(traceId), "Tablet.Transaction");
-        TxSpan.Attribute("Type", TypeName(*this));
+        if (TxSpan) {
+            TxSpan.Attribute("Type", TypeName(*this));
+        }
     }
 
 public: