wa-ooo commented on issue #3177:
URL: 
https://github.com/apache/incubator-dolphinscheduler/issues/3177#issuecomment-663987973


   > > > This parameter will only return the current timestamp. What you said 
should be the function of `supplementing data`.
   > > > For example, when a user needs to rerun the task of the previous day 
or supplement data of the previous day , the user just wants this parameter of 
the previous day's time. Is this the scene you are referring to?
   > > 
   > > 
   > > Yes, it is easier to use if you rerun the working instance directly 
without modifying the script
   > 
   > We can use this params.
   > 
![image](https://user-images.githubusercontent.com/29545877/88475457-872f7380-cf62-11ea-91e3-8d8047c65896.png)
   
    Yes, `Fix - 3177` make `ParameterUtils.convertParameterPlaceholders ()` 
method of using ${system.datetime} to parse the time expressions in the script 
(`$[yyyyMMdd]`),it can solve the problem. But I have a question, In 
`shelltask.java` in 151 lines at the same time use the `ParameterUtils. 
ConvertParameterPlaceholders2()` method to parse time expression ($[yyyyMMdd]), 
the code is always not be performed, because 
`ParameterUtils.ConvertParameterPlaceholders ()` method has been parsed.I think 
that in `ParameterUtils ConvertParameterPlaceholders2 ()` method is used the 
`schedule.time` to parse time expression is more reasonable


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to