diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/OSUtils.java b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/OSUtils.java
index 88f6c3970b27c7bede3f9b5662867769b08eb0f8..57190e84b8f742a994e5c4e37486a6dd27e8b62b 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/OSUtils.java
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/OSUtils.java
@@ -17,6 +17,9 @@
 
 package org.apache.dolphinscheduler.plugin.alert.script;
 
+/**
+ * OSUtils
+ */
 public class OSUtils {
 
     public static Boolean isWindows() {
diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ProcessUtils.java b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ProcessUtils.java
index d3e8fae3f5573c759e3ee6bd3135b7e47ee4b1d8..47d0d397e92c9904a96d4754f6c68198ce48cc70 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ProcessUtils.java
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ProcessUtils.java
@@ -22,12 +22,22 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * ProcessUtils
  */
 public class ProcessUtils {
 
+    private static final Logger logger = LoggerFactory.getLogger(ProcessUtils.class);
 
+    /**
+     * executeScript
+     *
+     * @param cmd cmd params
+     * @return exit code
+     */
     public static Integer executeScript(String... cmd) {
 
         int exitCode = -1;
@@ -49,7 +59,7 @@ public class ProcessUtils {
             return process.waitFor();
 
         } catch (IOException | InterruptedException e) {
-            e.printStackTrace();
+            logger.error("execute alert script error", e.getMessage());
         }
 
         return exitCode;
diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptAlertChannelFactory.java b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptAlertChannelFactory.java
index fc338625c13edac103aaf23335a4695f877059c5..2b04531a989e72a663d10e08518fab18316efcb8 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptAlertChannelFactory.java
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptAlertChannelFactory.java
@@ -17,9 +17,6 @@
 
 package org.apache.dolphinscheduler.plugin.alert.script;
 
-import java.util.Arrays;
-import java.util.List;
-
 import org.apache.dolphinscheduler.spi.alert.AlertChannel;
 import org.apache.dolphinscheduler.spi.alert.AlertChannelFactory;
 import org.apache.dolphinscheduler.spi.params.InputParam;
@@ -28,6 +25,9 @@ import org.apache.dolphinscheduler.spi.params.base.ParamsOptions;
 import org.apache.dolphinscheduler.spi.params.base.PluginParams;
 import org.apache.dolphinscheduler.spi.params.base.Validate;
 
+import java.util.Arrays;
+import java.util.List;
+
 /**
  * ScriptAlertChannelFactory
  */
diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptSender.java b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptSender.java
index df99a4f1a97556beb6ddfcdae734cc00b364225a..f7613873b40c9cda15ac569e48c82c8f8791f158 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptSender.java
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptSender.java
@@ -31,7 +31,6 @@ public class ScriptSender {
 
     private static final Logger logger = LoggerFactory.getLogger(ScriptSender.class);
 
-
     private String scriptPath;
 
     private Integer scriptType;
@@ -44,7 +43,6 @@ public class ScriptSender {
         userParams = config.get(ScriptParamsConstants.NAME_SCRIPT_USER_PARAMS);
     }
 
-
     public AlertResult sendScriptAlert(String msg) {
         AlertResult alertResult = new AlertResult();
         if (ScriptType.of(scriptType).equals(ScriptType.SHELL)) {
diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptType.java b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptType.java
index a7544cc46a9c1b64d46d3a550136b39d49623e1e..59b17d0a58de8b84f81d487c85ab65247e1d5468 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptType.java
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/ScriptType.java
@@ -53,7 +53,6 @@ public enum ScriptType {
         }
     }
 
-
     public static ScriptType of(Integer code) {
         if (SCRIPT_TYPE_MAP.containsKey(code)) {
             return SCRIPT_TYPE_MAP.get(code);
diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/StreamGobbler.java b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/StreamGobbler.java
index 13f8790cbc7de8792bdd692d469cfa0efe3f0348..1d8a516f37b75656349792cd12f9f899d7a8098e 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/StreamGobbler.java
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/main/java/org/apache/dolphinscheduler/plugin/alert/script/StreamGobbler.java
@@ -34,12 +34,10 @@ public class StreamGobbler extends Thread {
 
     InputStream inputStream;
 
-
     StreamGobbler(InputStream inputStream) {
         this.inputStream = inputStream;
     }
 
-
     public void run() {
         InputStreamReader inputStreamReader = new InputStreamReader(inputStream);
         BufferedReader inputBufferReader = new BufferedReader(inputStreamReader);
diff --git a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/test/script/shell/example.sh b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/test/script/shell/example.sh
index 6f215b011a6e34c45997977a931038729e87c0c9..708dcd004bcf9b841a8491b6ad52e3e8802740aa 100644
--- a/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/test/script/shell/example.sh
+++ b/dolphinscheduler-alert-plugin/dolphinscheduler-alert-script/src/test/script/shell/example.sh
@@ -15,14 +15,11 @@
 # limitations under the License.
 #
 
-content=$1
 
-msg=$2
+msg=$1
+content=$2
 
 # Write your specific logic here
-echo $content
-
-echo $msg
 
 # Set the exit code according to your execution result, and alert needs to use it to judge the status of this alarm result
-exit 888
+exit 0