[doris] 04/06: [fix](profile) fix profile sort child list exception (#17613)

[morningman]

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

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 62e7f6bee8ec1d4fef7dbf17e9456982c476af6f
Author: luozenglin <37725793+luozeng...@users.noreply.github.com>
AuthorDate: Fri Mar 10 08:44:32 2023 +0800

    [fix](profile) fix profile sort child list exception (#17613)
---
 .../src/main/java/org/apache/doris/common/util/RuntimeProfile.java | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/common/util/RuntimeProfile.java 
b/fe/fe-core/src/main/java/org/apache/doris/common/util/RuntimeProfile.java
index aceb7dc1e5..2f8fa9fa5d 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/common/util/RuntimeProfile.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/common/util/RuntimeProfile.java
@@ -439,6 +439,13 @@ public class RuntimeProfile {
         try {
             this.childList.sort((profile1, profile2) -> 
Long.compare(profile2.first.getCounterTotalTime().getValue(),
                     profile1.first.getCounterTotalTime().getValue()));
+        } catch (IllegalArgumentException e) {
+            // This exception may be thrown if the counter total time of the 
child is updated in the update method
+            // during the sorting process. This sorting only affects the 
profile instance display order, so this
+            // exception is temporarily ignored here.
+            if (LOG.isDebugEnabled()) {
+                LOG.debug("sort child list error: ", e);
+            }
         } finally {
             childLock.writeLock().unlock();
         }


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