bahman/scylla
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

stream_manager: update_progress: rename cf_id param to plan_id

Browse Source 
Before changing its type to streaming::plan_id
this patch clarifies that the parameter actually represents
the plan id and not the table id as its name suggests.

For reference, see the call to update_progress in
`stream_transfer_task::execute`, as well as the function
using _stream_bytes which map key is the plan id.

Signed-off-by: Benny Halevy <bhalevy@scylladb.com>
This commit is contained in:
Benny Halevy
2022-08-21 16:56:41 +03:00
parent c1fc0672a5
commit 3554533e2c
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
streaming/stream_manager.cc
View File

@@ -155,8 +155,8 @@ std::vector<shared_ptr<stream_result_future>> stream_manager::get_all_streams()
return result;
}
void stream_manager::update_progress(UUID cf_id, gms::inet_address peer, progress_info::direction dir, size_t fm_size) {
auto& sbytes = _stream_bytes[cf_id];
void stream_manager::update_progress(UUID plan_id, gms::inet_address peer, progress_info::direction dir, size_t fm_size) {
auto& sbytes = _stream_bytes[plan_id];
if (dir == progress_info::direction::OUT) {
sbytes[peer].bytes_sent += fm_size;
_total_outgoing_bytes += fm_size;

2
streaming/stream_manager.hh
View File

@@ -146,7 +146,7 @@ public:
return make_ready_future<>();
}
void update_progress(UUID cf_id, gms::inet_address peer, progress_info::direction dir, size_t fm_size);
void update_progress(UUID plan_id, gms::inet_address peer, progress_info::direction dir, size_t fm_size);
future<> update_all_progress_info();
void remove_progress(UUID plan_id);