morningman commented on a change in pull request #6196:
URL: https://github.com/apache/incubator-doris/pull/6196#discussion_r667009395



##########
File path: be/src/agent/task_worker_pool.cpp
##########
@@ -1150,6 +1150,9 @@ void 
TaskWorkerPool::_report_disk_state_worker_thread_callback() {
             disk.__set_disk_total_capacity(root_path_info.disk_capacity);
             disk.__set_data_used_capacity(root_path_info.data_used_capacity);
             disk.__set_disk_available_capacity(root_path_info.available);
+            std::filesystem::path trash_path(root_path_info.path + 
TRASH_PREFIX);
+            disk.__set_trash_used_capacity(

Review comment:
       Why not update the trash dir size in `DataDir::update_capacity()`, just 
like `data_used_capacity`?

##########
File path: fe/fe-core/src/main/java/org/apache/doris/common/util/DebugUtil.java
##########
@@ -27,7 +27,7 @@
 import java.util.UUID;
 
 public class DebugUtil {
-    public static final DecimalFormat DECIMAL_FORMAT_SCALE_3 = new 
DecimalFormat("#.000");
+    public static final DecimalFormat DECIMAL_FORMAT_SCALE_3 = new 
DecimalFormat("0.000");

Review comment:
       Why change this? Is it some thing wrong with origin `#.000`?




-- 
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

Reply via email to