Skip to content

Commit 34372a6

Browse files
authored
Tracing levels proposal implementation (#2415)
1 parent 2624e3d commit 34372a6

11 files changed

+93
-52
lines changed

ydb/core/keyvalue/keyvalue_intermediate.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ TIntermediate::TIntermediate(TActorId respondTo, TActorId keyValueActorId, ui64
8080
, CreatedAtGeneration(channelGeneration)
8181
, CreatedAtStep(channelStep)
8282
, IsReplied(false)
83-
, Span(TWilsonTablet::Tablet, std::move(traceId), "KeyValue.Intermediate", NWilson::EFlags::AUTO_END)
83+
, Span(TWilsonTablet::TabletTopLevel, std::move(traceId), "KeyValue.Intermediate", NWilson::EFlags::AUTO_END)
8484
{
8585
Stat.IntermediateCreatedAt = TAppData::TimeProvider->Now();
8686
Stat.RequestType = requestType;
@@ -106,7 +106,7 @@ void TIntermediate::UpdateStat() {
106106
}
107107
}
108108
} else {
109-
Stat.IndexRangeRead++;
109+
Stat.IndexRangeRead++;
110110
}
111111
};
112112

ydb/core/keyvalue/keyvalue_storage_read_request.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -511,7 +511,7 @@ class TKeyValueStorageReadRequest : public TActorBootstrapped<TKeyValueStorageRe
511511
: IntermediateResult(std::move(intermediate))
512512
, TabletInfo(const_cast<TTabletStorageInfo*>(tabletInfo))
513513
, TabletGeneration(tabletGeneration)
514-
, Span(TWilsonTablet::Tablet, IntermediateResult->Span.GetTraceId(), "KeyValue.StorageReadRequest")
514+
, Span(TWilsonTablet::TabletBasic, IntermediateResult->Span.GetTraceId(), "KeyValue.StorageReadRequest")
515515
{}
516516
};
517517

ydb/core/keyvalue/keyvalue_storage_request.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ class TKeyValueStorageRequest : public TActorBootstrapped<TKeyValueStorageReques
7878
, TabletGeneration(tabletGeneration)
7979
, IntermediateResults(std::move(intermediate))
8080
, TabletInfo(const_cast<TTabletStorageInfo*>(tabletInfo))
81-
, Span(TWilsonTablet::Tablet, IntermediateResults->Span.GetTraceId(), "KeyValue.StorageRequest")
81+
, Span(TWilsonTablet::TabletBasic, IntermediateResults->Span.GetTraceId(), "KeyValue.StorageRequest")
8282
{
8383
IntermediateResults->Stat.KeyvalueStorageRequestSentAt = TAppData::TimeProvider->Now();
8484
}

ydb/core/tablet/tablet_req_writelog.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ class TTabletReqWriteLog : public TActorBootstrapped<TTabletReqWriteLog> {
154154
, CommitTactic(commitTactic)
155155
, Info(info)
156156
, RepliesToWait(Max<ui32>())
157-
, RequestSpan(TWilsonTablet::Tablet, std::move(traceId), "Tablet.WriteLog")
157+
, RequestSpan(TWilsonTablet::TabletDetailed, std::move(traceId), "Tablet.WriteLog")
158158
{
159159
References.swap(refs);
160160
Y_ABORT_UNLESS(Info);
@@ -171,9 +171,9 @@ class TTabletReqWriteLog : public TActorBootstrapped<TTabletReqWriteLog> {
171171
NWilson::TTraceId innerTraceId;
172172

173173
if (RequestSpan) {
174-
auto res = BlobSpans.try_emplace(ref.Id, TWilsonTablet::Tablet, RequestSpan.GetTraceId(), "Tablet.WriteLog.Reference");
174+
auto res = BlobSpans.try_emplace(ref.Id, TWilsonTablet::TabletFull, RequestSpan.GetTraceId(), "Tablet.WriteLog.Reference");
175175

176-
innerTraceId = std::move(res.first->second.GetTraceId());
176+
innerTraceId = res.first->second.GetTraceId();
177177
}
178178

179179
SendToBS(ref.Id, ref.Buffer, ctx, handleClass, ref.Tactic ? *ref.Tactic : CommitTactic, std::move(innerTraceId));
@@ -191,7 +191,7 @@ class TTabletReqWriteLog : public TActorBootstrapped<TTabletReqWriteLog> {
191191
NWilson::TTraceId traceId;
192192

193193
if (RequestSpan) {
194-
auto res = BlobSpans.try_emplace(actualLogEntryId, TWilsonTablet::Tablet, RequestSpan.GetTraceId(), "Tablet.WriteLog.LogEntry");
194+
auto res = BlobSpans.try_emplace(actualLogEntryId, TWilsonTablet::TabletFull, RequestSpan.GetTraceId(), "Tablet.WriteLog.LogEntry");
195195

196196
traceId = std::move(res.first->second.GetTraceId());
197197
}

ydb/core/tablet_flat/flat_exec_seat.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,15 @@ namespace NTabletFlatExecutor {
3535
void Terminate(ETerminationReason reason, const TActorContext& ctx) noexcept;
3636

3737
void StartEnqueuedSpan() noexcept {
38-
WaitingSpan = NWilson::TSpan(TWilsonTablet::Tablet, Self->TxSpan.GetTraceId(), "Tablet.Transaction.Enqueued");
38+
WaitingSpan = NWilson::TSpan(TWilsonTablet::TabletDetailed, Self->TxSpan.GetTraceId(), "Tablet.Transaction.Enqueued");
3939
}
4040

4141
void FinishEnqueuedSpan() noexcept {
4242
WaitingSpan.EndOk();
4343
}
4444

4545
void CreatePendingSpan() noexcept {
46-
WaitingSpan = NWilson::TSpan(TWilsonTablet::Tablet, Self->TxSpan.GetTraceId(), "Tablet.Transaction.Pending");
46+
WaitingSpan = NWilson::TSpan(TWilsonTablet::TabletDetailed, Self->TxSpan.GetTraceId(), "Tablet.Transaction.Pending");
4747
}
4848

4949
void FinishPendingSpan() noexcept {

ydb/core/tablet_flat/flat_executor.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4234,7 +4234,7 @@ TString TExecutor::CheckBorrowConsistency() {
42344234

42354235
TTransactionWaitPad::TTransactionWaitPad(THolder<TSeat> seat)
42364236
: Seat(std::move(seat))
4237-
, WaitingSpan(NWilson::TSpan(TWilsonTablet::Tablet, Seat->GetTxTraceId(), "Tablet.Transaction.Wait"))
4237+
, WaitingSpan(NWilson::TSpan(TWilsonTablet::TabletDetailed, Seat->GetTxTraceId(), "Tablet.Transaction.Wait"))
42384238
{}
42394239

42404240
TTransactionWaitPad::~TTransactionWaitPad()

ydb/core/tablet_flat/tablet_flat_executor.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ class TTransactionContext : public TTxMemoryProviderBase {
229229
}
230230

231231
void StartExecutionSpan() noexcept {
232-
TransactionExecutionSpan = NWilson::TSpan(TWilsonTablet::Tablet, TransactionSpan.GetTraceId(), "Tablet.Transaction.Execute");
232+
TransactionExecutionSpan = NWilson::TSpan(TWilsonTablet::TabletDetailed, TransactionSpan.GetTraceId(), "Tablet.Transaction.Execute");
233233
}
234234

235235
void FinishExecutionSpan() noexcept {
@@ -289,7 +289,7 @@ class ITransaction : TNonCopyable {
289289
{ }
290290

291291
ITransaction(NWilson::TTraceId &&traceId)
292-
: TxSpan(NWilson::TSpan(TWilsonTablet::Tablet, std::move(traceId), "Tablet.Transaction"))
292+
: TxSpan(NWilson::TSpan(TWilsonTablet::TabletBasic, std::move(traceId), "Tablet.Transaction"))
293293
{ }
294294

295295
virtual ~ITransaction() = default;
@@ -314,7 +314,7 @@ class ITransaction : TNonCopyable {
314314
}
315315

316316
void SetupTxSpan(NWilson::TTraceId traceId) noexcept {
317-
TxSpan = NWilson::TSpan(TWilsonTablet::Tablet, std::move(traceId), "Tablet.Transaction");
317+
TxSpan = NWilson::TSpan(TWilsonTablet::TabletBasic, std::move(traceId), "Tablet.Transaction");
318318
if (TxSpan) {
319319
TxSpan.Attribute("Type", TypeName(*this));
320320
}

ydb/core/tx/datashard/datashard.cpp

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2905,8 +2905,10 @@ void TDataShard::ProposeTransaction(TEvDataShard::TEvProposeTransaction::TPtr &&
29052905
UpdateProposeQueueSize();
29062906
} else {
29072907
// Prepare planned transactions as soon as possible
2908-
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::Tablet, std::move(ev->TraceId), "Datashard.Transaction", NWilson::EFlags::AUTO_END);
2909-
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
2908+
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::TabletTopLevel, std::move(ev->TraceId), "Datashard.Transaction", NWilson::EFlags::AUTO_END);
2909+
if (datashardTransactionSpan) {
2910+
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
2911+
}
29102912

29112913
Execute(new TTxProposeTransactionBase(this, std::move(ev), TAppData::TimeProvider->Now(), NextTieBreakerIndex++, /* delayed */ false, std::move(datashardTransactionSpan)), ctx);
29122914
}
@@ -2926,8 +2928,10 @@ void TDataShard::ProposeTransaction(NEvents::TDataEvents::TEvWrite::TPtr&& ev, c
29262928
UpdateProposeQueueSize();
29272929
} else {
29282930
// Prepare planned transactions as soon as possible
2929-
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::Tablet, std::move(ev->TraceId), "Datashard.WriteTransaction", NWilson::EFlags::AUTO_END);
2930-
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
2931+
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::TabletTopLevel, std::move(ev->TraceId), "Datashard.WriteTransaction", NWilson::EFlags::AUTO_END);
2932+
if (datashardTransactionSpan) {
2933+
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
2934+
}
29312935

29322936
Execute(new TTxWrite(this, std::move(ev), TAppData::TimeProvider->Now(), NextTieBreakerIndex++, /* delayed */ false, std::move(datashardTransactionSpan)), ctx);
29332937
}
@@ -2994,16 +2998,20 @@ void TDataShard::Handle(TEvPrivate::TEvDelayedProposeTransaction::TPtr &ev, cons
29942998
switch (item.Event->GetTypeRewrite()) {
29952999
case TEvDataShard::TEvProposeTransaction::EventType: {
29963000
auto event = IEventHandle::Downcast<TEvDataShard::TEvProposeTransaction>(std::move(item.Event));
2997-
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::Tablet, std::move(event->TraceId), "Datashard.Transaction", NWilson::EFlags::AUTO_END);
2998-
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
3001+
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::TabletTopLevel, std::move(event->TraceId), "Datashard.Transaction", NWilson::EFlags::AUTO_END);
3002+
if (datashardTransactionSpan) {
3003+
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
3004+
}
29993005

30003006
Execute(new TTxProposeTransactionBase(this, std::move(event), item.ReceivedAt, item.TieBreakerIndex, /* delayed */ true, std::move(datashardTransactionSpan)), ctx);
30013007
return;
30023008
}
30033009
case NEvents::TDataEvents::TEvWrite::EventType: {
30043010
auto event = IEventHandle::Downcast<NEvents::TDataEvents::TEvWrite>(std::move(item.Event));
3005-
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::Tablet, std::move(event->TraceId), "Datashard.WriteTransaction", NWilson::EFlags::AUTO_END);
3006-
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
3011+
NWilson::TSpan datashardTransactionSpan(TWilsonTablet::TabletTopLevel, std::move(event->TraceId), "Datashard.WriteTransaction", NWilson::EFlags::AUTO_END);
3012+
if (datashardTransactionSpan) {
3013+
datashardTransactionSpan.Attribute("Shard", std::to_string(TabletID()));
3014+
}
30073015

30083016
Execute(new TTxWrite(this, std::move(event), item.ReceivedAt, item.TieBreakerIndex, /* delayed */ true, std::move(datashardTransactionSpan)), ctx);
30093017
return;

ydb/core/tx/datashard/datashard__read_iterator.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2545,7 +2545,7 @@ void TDataShard::Handle(TEvDataShard::TEvRead::TPtr& ev, const TActorContext& ct
25452545
auto* request = ev->Get();
25462546

25472547
if (!request->ReadSpan) {
2548-
request->ReadSpan = NWilson::TSpan(TWilsonTablet::Tablet, std::move(ev->TraceId), "Datashard.Read", NWilson::EFlags::AUTO_END);
2548+
request->ReadSpan = NWilson::TSpan(TWilsonTablet::TabletTopLevel, std::move(ev->TraceId), "Datashard.Read", NWilson::EFlags::AUTO_END);
25492549
request->ReadSpan.Attribute("Shard", std::to_string(TabletID()));
25502550
}
25512551

ydb/core/tx/datashard/datashard_pipeline.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1783,7 +1783,7 @@ EExecutionStatus TPipeline::RunExecutionPlan(TOperation::TPtr op,
17831783
return EExecutionStatus::Reschedule;
17841784
}
17851785

1786-
NWilson::TSpan unitSpan(TWilsonTablet::Tablet, txc.TransactionExecutionSpan.GetTraceId(), "Datashard.Unit");
1786+
NWilson::TSpan unitSpan(TWilsonTablet::TabletFull, txc.TransactionExecutionSpan.GetTraceId(), "Datashard.Unit");
17871787

17881788
NCpuTime::TCpuTimer timer;
17891789
auto status = unit.Execute(op, txc, ctx);

0 commit comments

Comments
 (0)