You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by zi...@apache.org on 2022/12/09 08:17:28 UTC

[dolphinscheduler] branch dev updated: Throws the correct exception for the updateResource interface (#13120)

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

zihaoxiang pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git


The following commit(s) were added to refs/heads/dev by this push:
     new f9cf9331a6 Throws the correct exception for the updateResource interface (#13120)
f9cf9331a6 is described below

commit f9cf9331a6d9399d512302dead39880fcd59d3e2
Author: J·Y <ji...@gmail.com>
AuthorDate: Fri Dec 9 16:17:20 2022 +0800

    Throws the correct exception for the updateResource interface (#13120)
    
    Co-authored-by: rqyin <rq...@easipass.com>
---
 .../apache/dolphinscheduler/api/service/impl/ResourcesServiceImpl.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ResourcesServiceImpl.java b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ResourcesServiceImpl.java
index 694bb1dfb3..4cc20c7baf 100644
--- a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ResourcesServiceImpl.java
+++ b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ResourcesServiceImpl.java
@@ -72,7 +72,6 @@ import org.apache.commons.lang3.StringUtils;
 
 import java.io.File;
 import java.io.IOException;
-import java.rmi.ServerException;
 import java.text.MessageFormat;
 import java.time.LocalDateTime;
 import java.time.temporal.ChronoUnit;
@@ -1739,7 +1738,7 @@ public class ResourcesServiceImpl extends BaseServiceImpl implements ResourcesSe
         } catch (IOException e) {
             logger.error("Download resource error, the path is {}, and local filename is {}, the error message is {}",
                     fullName, localFileName, e.getMessage());
-            throw new ServerException("Download the resource file failed ,it may be related to your storage");
+            throw new ServiceException("Download the resource file failed ,it may be related to your storage");
         }
     }