[Fix-13814] [Built-in variable] shell task transfer param (#13974)

dolphinscheduler-spi/src/main/java/org/apache/dolphinscheduler/spi/task/paramparser/ParamUtils.java

@@ -69,7 +69,7 @@ public class ParamUtils {
 
         Map<String, Property> varParams = parameters.getVarPoolMap();
 
-        if (MapUtils.isEmpty(globalParams) && MapUtils.isEmpty(localParams)) {
+        if (MapUtils.isEmpty(globalParams) && MapUtils.isEmpty(localParams) && MapUtils.isEmpty(varParams)) {
             return null;
         }
         // if it is a complement,
@@ -98,7 +98,9 @@ public class ParamUtils {
             convertedParams = localParams;
         }
         if (varParams != null) {
-            varParams.putAll(globalParams);
+            if (globalParams != null) {
+                varParams.putAll(globalParams);
+            }
             globalParams = varParams;
             for (Map.Entry<String, Property> entry : varParams.entrySet()) {
                 convertedParams.put(entry.getKey(), entry.getValue());

dolphinscheduler-task-plugin/dolphinscheduler-task-sql/src/main/java/org/apache/dolphinscheduler/plugin/task/sql/SqlTask.java

@@ -436,6 +436,8 @@ public class SqlTask extends AbstractTaskExecutor {
         Map<Integer, Property> sqlParamsMap = new HashMap<>();
         StringBuilder sqlBuilder = new StringBuilder();
 
+        //replace variable TIME with $[YYYYmmddd...] in sql when history run job and batch complement job
+        sql = ParameterUtils.replaceScheduleTime(sql, taskExecutionContext.getScheduleTime());
         // combining local and global parameters
         Map<String, Property> paramsMap = ParamUtils.convert(taskExecutionContext, getParameters());
 
@@ -452,9 +454,6 @@ public class SqlTask extends AbstractTaskExecutor {
             sqlParameters.setTitle(title);
         }
 
-        //new
-        //replace variable TIME with $[YYYYmmddd...] in sql when history run job and batch complement job
-        sql = ParameterUtils.replaceScheduleTime(sql, taskExecutionContext.getScheduleTime());
         // special characters need to be escaped, ${} needs to be escaped
         setSqlParamsMap(sql, rgex, sqlParamsMap, paramsMap,taskExecutionContext.getTaskInstanceId());
         //Replace the original value in sql ！{...} ，Does not participate in precompilation