Skip to content

Commit 3170fe5

Browse files
authored
LocalDB: rename DataCleanup to Vacuum (#19993)
1 parent c5affc5 commit 3170fe5

16 files changed

+308
-308
lines changed

ydb/core/keyvalue/keyvalue_flat_impl.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -566,7 +566,7 @@ class TKeyValueFlat : public TActor<TKeyValueFlat>, public NTabletFlatExecutor::
566566
void Handle(TEvKeyValue::TEvForceTabletDataCleanup::TPtr &ev) {
567567
ALOG_DEBUG(NKikimrServices::KEYVALUE, "KeyValue# " << TabletID()
568568
<< " Handle TEvForceTabletDataCleanup generation# " << ev->Get()->Generation);
569-
Executor()->CleanupData(ev->Get()->Generation);
569+
Executor()->StartVacuum(ev->Get()->Generation);
570570
}
571571

572572
public:
@@ -609,8 +609,8 @@ class TKeyValueFlat : public TActor<TKeyValueFlat>, public NTabletFlatExecutor::
609609
return false;
610610
}
611611

612-
void DataCleanupComplete(ui64 dataCleanupGeneration, const TActorContext &ctx) override {
613-
STLOG(NLog::PRI_DEBUG, NKikimrServices::KEYVALUE_GC, KV271, "DataCleanupComplete",
612+
void VacuumComplete(ui64 dataCleanupGeneration, const TActorContext &ctx) override {
613+
STLOG(NLog::PRI_DEBUG, NKikimrServices::KEYVALUE_GC, KV271, "VacuumComplete",
614614
(TabletId, TabletID()));
615615
Execute(new TTxCompleteCleanupData(this, State.GetCleanupResetGeneration(), dataCleanupGeneration), ctx);
616616
}

ydb/core/tablet_flat/flat_executor.cpp

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -472,7 +472,7 @@ void TExecutor::Active(const TActorContext &ctx) {
472472

473473
CompactionLogic = THolder<TCompactionLogic>(new TCompactionLogic(MemTableMemoryConsumersCollection.Get(), Logger.Get(), Broker.Get(), this, loadedState->Comp,
474474
Sprintf("tablet-%" PRIu64, Owner->TabletID())));
475-
DataCleanupLogic = MakeHolder<TDataCleanupLogic>(static_cast<NActors::IActorOps*>(this), this, Owner, Logger.Get(), GcLogic.Get());
475+
VacuumLogic = MakeHolder<TVacuumLogic>(static_cast<NActors::IActorOps*>(this), this, Owner, Logger.Get(), GcLogic.Get());
476476
LogicRedo->InstallCounters(Counters.Get(), AppTxCounters);
477477

478478
ResourceMetrics = MakeHolder<NMetrics::TResourceMetrics>(Owner->TabletID(), 0, Launcher);
@@ -812,7 +812,7 @@ TExecutorCaches TExecutor::CleanupState() {
812812
Y_ENSURE(!LogicAlter);
813813
Y_ENSURE(!CompactionLogic);
814814
BorrowLogic.Destroy();
815-
DataCleanupLogic.Destroy();
815+
VacuumLogic.Destroy();
816816

817817
return caches;
818818
}
@@ -996,7 +996,7 @@ void TExecutor::CheckCollectionBarrier(TIntrusivePtr<TBarrier> &barrier) {
996996
Owner->CompletedLoansChanged(OwnerCtx());
997997
}
998998
}
999-
if (DataCleanupLogic->NeedGC()) {
999+
if (VacuumLogic->NeedGC()) {
10001000
GcLogic->SendCollectGarbage(ActorContext());
10011001
}
10021002
}
@@ -1418,7 +1418,7 @@ void TExecutor::Handle(TEvTablet::TEvGcForStepAckResponse::TPtr &ev) {
14181418
return;
14191419
}
14201420

1421-
DataCleanupLogic->OnGcForStepAckResponse(Generation(), ev->Get()->Step, OwnerCtx());
1421+
VacuumLogic->OnGcForStepAckResponse(Generation(), ev->Get()->Step, OwnerCtx());
14221422
}
14231423

14241424
void TExecutor::AdvancePendingPartSwitches() {
@@ -1433,8 +1433,8 @@ void TExecutor::AdvancePendingPartSwitches() {
14331433
PlanTransactionActivation();
14341434
MaybeRelaxRejectProbability();
14351435

1436-
// Note: followers don't have DataCleanupLogic
1437-
if (NeedFollowerSnapshot || DataCleanupLogic && DataCleanupLogic->NeedLogSnaphot()) {
1436+
// Note: followers don't have VacuumLogic
1437+
if (NeedFollowerSnapshot || VacuumLogic && VacuumLogic->NeedLogSnaphot()) {
14381438
MakeLogSnapshot();
14391439
}
14401440
}
@@ -2872,7 +2872,7 @@ void TExecutor::MakeLogSnapshot() {
28722872
BorrowLogic->SnapToLog(snap, *commit);
28732873
GcLogic->SnapToLog(snap, commit->Step);
28742874
LogicSnap->MakeSnap(snap, *commit, Logger.Get());
2875-
DataCleanupLogic->OnMakeLogSnapshot(Generation(), commit->Step);
2875+
VacuumLogic->OnMakeLogSnapshot(Generation(), commit->Step);
28762876

28772877
AttachLeaseCommit(commit.Get(), /* force */ true);
28782878
CommitManager->Commit(commit);
@@ -3187,8 +3187,8 @@ void TExecutor::Handle(TEvTablet::TEvCommitResult::TPtr &ev, const TActorContext
31873187
case ECommit::Snap:
31883188
LogicSnap->Confirm(msg->Step);
31893189

3190-
DataCleanupLogic->OnSnapshotCommited(Generation(), step);
3191-
if (NeedFollowerSnapshot || DataCleanupLogic->NeedLogSnaphot())
3190+
VacuumLogic->OnSnapshotCommited(Generation(), step);
3191+
if (NeedFollowerSnapshot || VacuumLogic->NeedLogSnaphot())
31923192
MakeLogSnapshot();
31933193

31943194
break;
@@ -3227,7 +3227,7 @@ void TExecutor::Handle(TEvBlobStorage::TEvCollectGarbageResult::TPtr &ev) {
32273227
if (auto retryDelay = GcLogic->OnCollectGarbageResult(ev)) {
32283228
Schedule(retryDelay, new TEvPrivate::TEvRetryGcRequest(ev->Get()->Channel));
32293229
}
3230-
DataCleanupLogic->OnCollectedGarbage(OwnerCtx());
3230+
VacuumLogic->OnCollectedGarbage(OwnerCtx());
32313231
}
32323232

32333233
void TExecutor::Handle(TEvPrivate::TEvRetryGcRequest::TPtr &ev, const TActorContext &ctx) {
@@ -3747,7 +3747,7 @@ void TExecutor::Handle(NOps::TEvResult *ops, TProdCompact *msg, bool cancelled)
37473747

37483748
Y_ENSURE(InFlyCompactionGcBarriers.emplace(commit->Step, ops->Barrier).second);
37493749

3750-
DataCleanupLogic->OnCompleteCompaction(tableId, CompactionLogic->GetFinishedCompactionInfo(tableId));
3750+
VacuumLogic->OnCompleteCompaction(tableId, CompactionLogic->GetFinishedCompactionInfo(tableId));
37513751

37523752
AttachLeaseCommit(commit.Get());
37533753
CommitManager->Commit(commit);
@@ -3780,7 +3780,7 @@ void TExecutor::Handle(NOps::TEvResult *ops, TProdCompact *msg, bool cancelled)
37803780

37813781
activeTransaction.Done();
37823782

3783-
if (LogicSnap->MayFlush(false) || DataCleanupLogic->NeedLogSnaphot()) {
3783+
if (LogicSnap->MayFlush(false) || VacuumLogic->NeedLogSnaphot()) {
37843784
MakeLogSnapshot();
37853785
}
37863786
}
@@ -4143,14 +4143,14 @@ bool TExecutor::CompactTables() {
41434143
}
41444144
}
41454145

4146-
void TExecutor::CleanupData(ui64 dataCleanupGeneration) {
4147-
if (DataCleanupLogic->TryStartCleanup(dataCleanupGeneration, OwnerCtx())) {
4146+
void TExecutor::StartVacuum(ui64 vacuumGeneration) {
4147+
if (VacuumLogic->TryStartVacuum(vacuumGeneration, OwnerCtx())) {
41484148
for (const auto& [tableId, _] : Scheme().Tables) {
41494149
auto compactionId = CompactionLogic->PrepareForceCompaction(tableId);
4150-
DataCleanupLogic->OnCompactionPrepared(tableId, compactionId);
4150+
VacuumLogic->OnCompactionPrepared(tableId, compactionId);
41514151
}
4152-
DataCleanupLogic->WaitCompaction();
4153-
if (DataCleanupLogic->NeedLogSnaphot()) {
4152+
VacuumLogic->WaitCompaction();
4153+
if (VacuumLogic->NeedLogSnaphot()) {
41544154
MakeLogSnapshot();
41554155
}
41564156
}

ydb/core/tablet_flat/flat_executor.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@
1414
#include "flat_exec_commit.h"
1515
#include "flat_executor_misc.h"
1616
#include "flat_executor_compaction_logic.h"
17-
#include "flat_executor_data_cleanup_logic.h"
1817
#include "flat_executor_gclogic.h"
18+
#include "flat_executor_vacuum_logic.h"
1919
#include "flat_bio_events.h"
2020
#include "flat_bio_stats.h"
2121
#include "flat_fwd_sieve.h"
@@ -467,7 +467,7 @@ class TExecutor
467467
THolder<TExecutorGCLogic> GcLogic;
468468
THolder<TCompactionLogic> CompactionLogic;
469469
THolder<TExecutorBorrowLogic> BorrowLogic;
470-
THolder<TDataCleanupLogic> DataCleanupLogic;
470+
THolder<TVacuumLogic> VacuumLogic;
471471

472472
TLoadBlobQueue PendingBlobQueue;
473473

@@ -683,7 +683,7 @@ class TExecutor
683683
ui64 CompactTable(ui32 tableId) override;
684684
bool CompactTables() override;
685685

686-
void CleanupData(ui64 dataCleanupGeneration) override;
686+
void StartVacuum(ui64 vacuumGeneration) override;
687687

688688
void Handle(NMemory::TEvMemTableRegistered::TPtr &ev);
689689
void Handle(NMemory::TEvMemTableCompact::TPtr &ev);

ydb/core/tablet_flat/flat_executor_data_cleanup_logic.cpp

Lines changed: 0 additions & 206 deletions
This file was deleted.

ydb/core/tablet_flat/flat_executor_ut.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -447,8 +447,8 @@ class TTestFlatTablet : public TActor<TTestFlatTablet>, public TTabletExecutedFl
447447
Send(Sender, new NFake::TEvCompacted(table));
448448
}
449449

450-
void DataCleanupComplete(ui64 dataCleanupComplete, const TActorContext&) override {
451-
Send(Sender, new NFake::TEvDataCleaned(dataCleanupComplete));
450+
void VacuumComplete(ui64 vacuumComplete, const TActorContext&) override {
451+
Send(Sender, new NFake::TEvDataCleaned(vacuumComplete));
452452
}
453453

454454
void ScanComplete(NTable::EStatus, TAutoPtr<IDestructable>, ui64 cookie, const TActorContext&) override

0 commit comments

Comments
 (0)