(doris) 25/44: [profile](fe)update format of min/hour/sec unit in profile (#37010)

[dataroaring]

This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git

commit ee516656cf447188b51c8d245c26ecfef4e9b247
Author: minghong <engle...@gmail.com>
AuthorDate: Mon Jul 1 16:27:25 2024 +0800

    [profile](fe)update format of min/hour/sec unit in profile (#37010)
    
    ## Proposed changes
    it helps us to find the point in profile which operator consumes long
    time
    
    Issue Number: close #xxx
    
    <!--Describe your changes.-->
---
 .../src/main/java/org/apache/doris/common/util/DebugUtil.java     | 6 +++---
 .../src/test/java/org/apache/doris/common/util/DebugUtilTest.java | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/common/util/DebugUtil.java 
b/fe/fe-core/src/main/java/org/apache/doris/common/util/DebugUtil.java
index 2a52420a96d..937c74cac66 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/common/util/DebugUtil.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/common/util/DebugUtil.java
@@ -71,17 +71,17 @@ public class DebugUtil {
             boolean hour = false;
             boolean minute = false;
             if (newValue >= HOUR) {
-                builder.append(newValue / HOUR).append("h");
+                builder.append(newValue / HOUR).append("hour");
                 newValue %= HOUR;
                 hour = true;
             }
             if (newValue >= MINUTE) {
-                builder.append(newValue / MINUTE).append("m");
+                builder.append(newValue / MINUTE).append("min");
                 newValue %= MINUTE;
                 minute = true;
             }
             if (!hour && newValue >= SECOND) {
-                builder.append(newValue / SECOND).append("s");
+                builder.append(newValue / SECOND).append("sec");
                 newValue %= SECOND;
             }
             if (!hour && !minute) {
diff --git 
a/fe/fe-core/src/test/java/org/apache/doris/common/util/DebugUtilTest.java 
b/fe/fe-core/src/test/java/org/apache/doris/common/util/DebugUtilTest.java
index 54a3f4c388b..aa599783f18 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/common/util/DebugUtilTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/common/util/DebugUtilTest.java
@@ -50,16 +50,16 @@ public class DebugUtilTest {
     @Test
     public void testGetPrettyStringMs() {
         // 6hour1min
-        Assert.assertEquals(DebugUtil.getPrettyStringMs(21660222), "6h1m");
+        Assert.assertEquals("6hour1min", 
DebugUtil.getPrettyStringMs(21660222));
 
         // 1min222ms
-        Assert.assertEquals(DebugUtil.getPrettyStringMs(60222), "1m");
+        Assert.assertEquals("1min", DebugUtil.getPrettyStringMs(60222));
 
         // 2s222ms
-        Assert.assertEquals(DebugUtil.getPrettyStringMs(2222), "2s222ms");
+        Assert.assertEquals("2sec222ms", DebugUtil.getPrettyStringMs(2222));
 
         // 22ms
-        Assert.assertEquals(DebugUtil.getPrettyStringMs(22), "22ms");
+        Assert.assertEquals("22ms", DebugUtil.getPrettyStringMs(22));
     }
 
     @Test


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org