Improve the Backend's disk info report performance #349
Conversation
1. Use the data size save in tablet's header to calculate the disk used capacity. 2. Decrease the default interval of disk and tablet report, from 10 min to 1 min.
| info.capacity = 1; | ||
| info.data_used_capacity = 0; | ||
| info.available = 0; | ||
| _get_path_available_capacity(info.path, &info.available); |
There was a problem hiding this comment.
If you get the value capacity, it may be better to get available by capacity minus data_used_capacity?
There was a problem hiding this comment.
For now, available capacity does not equal to the capacity - data_used_capacity;
Actually, capacity = data_used_capacity + available_capacity + other_used_capacity.
other_used_capacity includes logs/, trash/, mini_download/, meta/, etc, which are not included in data_used_capacity.
| path_map.emplace(path, it.second->to_root_path_info()); | ||
| // if this path is not used, init it's info | ||
| if (!path_map[path].is_used) { | ||
| path_map[path].capacity = 1; |
There was a problem hiding this comment.
It seems that value of capacity variable is not updated?
There was a problem hiding this comment.
Capacity is set in Store.cpp. And I think the capacity will not change. If it changes, the Backend process must be restarted and the capacity is set when store is being initialized.
4 participants
ISSUE: #348