github-actions[bot] commented on code in PR #27648:
URL: https://github.com/apache/doris/pull/27648#discussion_r1416627201
##########
be/src/olap/storage_engine.cpp:
##########
@@ -1424,4 +1426,24 @@ Status StorageEngine::_persist_broken_paths() {
return Status::OK();
}
+bool StorageEngine::_increase_low_priority_task_nums(DataDir* dir) {
Review Comment:
warning: method '_increase_low_priority_task_nums' can be made static
[readability-convert-member-functions-to-static]
be/src/olap/storage_engine.h:337:
```diff
- bool _increase_low_priority_task_nums(DataDir* dir);
+ static bool _increase_low_priority_task_nums(DataDir* dir);
```
##########
be/src/olap/storage_engine.cpp:
##########
@@ -1424,4 +1426,24 @@
return Status::OK();
}
+bool StorageEngine::_increase_low_priority_task_nums(DataDir* dir) {
+ if (!config::enable_compaction_priority_scheduling) {
+ return false;
+ }
+ std::lock_guard l(_low_priority_task_nums_mutex);
+ if (_low_priority_task_nums[dir] <
config::low_priority_compaction_task_num_per_disk) {
+ _low_priority_task_nums[dir]++;
+ return true;
+ }
+ return false;
+}
+
+void StorageEngine::_decrease_low_priority_task_nums(DataDir* dir) {
Review Comment:
warning: method '_decrease_low_priority_task_nums' can be made static
[readability-convert-member-functions-to-static]
be/src/olap/storage_engine.h:339:
```diff
- void _decrease_low_priority_task_nums(DataDir* dir);
+ static void _decrease_low_priority_task_nums(DataDir* dir);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
