Skip to content

Commit d101bb5

Browse files
committed
Add CostModel for ErasureNone, fix formula for write cost
1 parent 6f77ce8 commit d101bb5

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

ydb/core/blobstorage/vdisk/common/blobstorage_cost_tracker.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ TBsCostTracker::TBsCostTracker(const TBlobStorageGroupType& groupType, NPDisk::E
5050
CostModel = std::make_unique<TBsCostModelMirror3of4>(diskType);
5151
break;
5252
default:
53+
CostModel = std::make_unique<TBsCostModelErasureNone>(diskType);
5354
break;
5455
}
5556
}

ydb/core/blobstorage/vdisk/common/blobstorage_cost_tracker.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ class TBsCostModelBase {
100100
return HDDEstimator.Write(chunkSize);
101101
}
102102
default: {
103-
ui64 seekTime = 1. * chunkSize * DeviceSeekTimeNs;
103+
ui64 seekTime = DeviceSeekTimeNs / 100u; // assume we do one seek per 100 log records
104104
ui64 writeTime = chunkSize * 1'000'000'000ull / DeviceWriteSpeedBps;
105105
return seekTime + writeTime;
106106
}
@@ -253,6 +253,7 @@ class TBsCostModelBase {
253253
}
254254
};
255255

256+
using TBsCostModelErasureNone = TBsCostModelBase;
256257
class TBsCostModelMirror3dc;
257258
class TBsCostModel4Plus2Block;
258259
class TBsCostModelMirror3of4;

0 commit comments

Comments
 (0)