Skip to content

Commit d875083

Browse files
fix
1 parent 3a71fc5 commit d875083

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

ydb/core/tx/columnshard/hooks/abstract/abstract.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ ui64 ICSController::GetGuaranteeIndexationStartBytesLimit() const {
2424
return DoGetGuaranteeIndexationStartBytesLimit(defaultValue);
2525
}
2626

27-
bool ICSController::CheckPortionForEvict(const std::shared_ptr<NOlap::TPortionInfo>& portion) const {
28-
return portion->HasRuntimeFeature(NOlap::TPortionInfo::ERuntimeFeature::Optimized) && !portion->HasInsertWriteId();
27+
bool ICSController::CheckPortionForEvict(const NOlap::TPortionInfo& portion) const {
28+
return portion.HasRuntimeFeature(NOlap::TPortionInfo::ERuntimeFeature::Optimized) && !portion.HasInsertWriteId();
2929
}
3030

3131
}

ydb/core/tx/columnshard/hooks/abstract/abstract.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ class ICSController {
146146
const std::set<NOlap::TSnapshot>& /*snapshotsToSave*/, const std::set<NOlap::TSnapshot>& /*snapshotsToRemove*/) {
147147
}
148148

149-
virtual bool CheckPortionForEvict(const std::shared_ptr<NOlap::TPortionInfo>& portion) const;
149+
virtual bool CheckPortionForEvict(const NOlap::TPortionInfo& portion) const;
150150

151151
TDuration GetPingCheckPeriod() const {
152152
const TDuration defaultValue = 0.6 * GetReadTimeoutClean();

ydb/core/tx/columnshard/test_helper/controllers.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ class TWaitCompactionController: public NYDBTest::NColumnShard::TController {
3636
return TDuration::Zero();
3737
}
3838
public:
39-
virtual bool CheckPortionForEvict(const std::shared_ptr<TPortionInfo>& portion) const override {
39+
virtual bool CheckPortionForEvict(const TPortionInfo& portion) const override {
4040
if (SkipSpecialCheckForEvict) {
4141
return true;
4242
} else {

0 commit comments

Comments
 (0)