Skip to content

Commit e935dfa

Browse files
committed
cr fixes
1 parent 4aaee8c commit e935dfa

File tree

2 files changed

+5
-12
lines changed

2 files changed

+5
-12
lines changed

ydb/core/tablet_flat/flat_part_charge_btree_index.h

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -47,20 +47,13 @@ class TChargeBTreeIndex : public ICharge {
4747
, Env(env)
4848
{
4949
Y_UNUSED(includeHistory);
50-
// if (includeHistory && Part->HistoricGroupsCount) {
51-
// HistoryIndex.emplace(Part, Env, TGroupId(0, true));
52-
// }
5350

5451
TDynBitMap seen;
5552
for (TTag tag : tags) {
5653
if (const auto* col = Scheme.FindColumnByTag(tag)) {
5754
if (col->Group != 0 && !seen.Get(col->Group)) {
5855
NPage::TGroupId groupId(col->Group);
5956
Groups.push_back(groupId);
60-
// if (HistoryIndex) {
61-
// NPage::TGroupId historyGroupId(col->Group, true);
62-
// HistoryGroups.emplace_back(TPartIndexIt(Part, Env, historyGroupId), historyGroupId);
63-
// }
6457
seen.Set(col->Group);
6558
}
6659
}

ydb/core/tablet_flat/ut/ut_btree_index_iter_charge.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,15 @@ namespace {
7171
}
7272
}
7373

74-
struct MakePartParams {
74+
struct TMakePartParams {
7575
const ui32 Levels = Max<ui32>();
7676
const bool Groups = false;
7777
const bool History = false;
7878
const bool Slices = false;
7979
const ui32 Rows = 40;
8080
};
8181

82-
TPartEggs MakePart(MakePartParams params) {
82+
TPartEggs MakePart(TMakePartParams params) {
8383
NPage::TConf conf;
8484
switch (params.Levels) {
8585
case 0:
@@ -362,7 +362,7 @@ Y_UNIT_TEST_SUITE(TPartBtreeIndexIt) {
362362
}
363363
}
364364

365-
void CheckPart(MakePartParams params) {
365+
void CheckPart(TMakePartParams params) {
366366
TPartEggs eggs = MakePart(params);
367367
const auto part = *eggs.Lone();
368368

@@ -464,7 +464,7 @@ Y_UNIT_TEST_SUITE(TChargeBTreeIndex) {
464464
}
465465
}
466466

467-
void CheckPart(MakePartParams params) {
467+
void CheckPart(TMakePartParams params) {
468468
TPartEggs eggs = MakePart(params);
469469
const auto part = *eggs.Lone();
470470

@@ -646,7 +646,7 @@ Y_UNIT_TEST_SUITE(TPartBtreeIndexIteration) {
646646
}
647647
}
648648

649-
void CheckPart(MakePartParams params) {
649+
void CheckPart(TMakePartParams params) {
650650
TPartEggs eggs = MakePart(params);
651651
const auto part = *eggs.Lone();
652652

0 commit comments

Comments
 (0)