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

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


The following commit(s) were added to refs/heads/master by this push:
     new fa8c3aec07e [opt](load) catch Throwable to make load error msg more 
clear (#26821)
fa8c3aec07e is described below

commit fa8c3aec07ebc481638e2ae6fc09d20e5a49e8e8
Author: Mingyu Chen <morning...@163.com>
AuthorDate: Mon Nov 13 09:39:29 2023 +0800

    [opt](load) catch Throwable to make load error msg more clear (#26821)
    
    When doing LoadPendingTask or LoadLoadingTask, there may be some Error 
thrown,
    such as `NoClassDefFoundError`, but previously, we only catch java's 
`Exception`, so
    other kind of error can not be shown clearly.
---
 fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadTask.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadTask.java 
b/fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadTask.java
index dbedb9facc1..d6789805e67 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadTask.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadTask.java
@@ -91,10 +91,10 @@ public abstract class LoadTask extends MasterTask {
             failMsg.setMsg(e.getMessage() == null ? "" : e.getMessage());
             LOG.warn(new LogBuilder(LogKey.LOAD_JOB, callback.getCallbackId())
                     .add("error_msg", "Failed to execute load task").build(), 
e);
-        } catch (Exception e) {
-            failMsg.setMsg(e.getMessage() == null ? "" : e.getMessage());
+        } catch (Throwable t) {
+            failMsg.setMsg(t.getMessage() == null ? "" : t.getMessage());
             LOG.warn(new LogBuilder(LogKey.LOAD_JOB, callback.getCallbackId())
-                    .add("error_msg", "Unexpected failed to execute load 
task").build(), e);
+                    .add("error_msg", "Unexpected failed to execute load 
task").build(), t);
         } finally {
             if (!isFinished) {
                 // callback on pending task failed


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

Reply via email to