reele commented on issue #15129:
URL: 
https://github.com/apache/dolphinscheduler/issues/15129#issuecomment-1802415738

   > Good catch! Would you like to submit a PR to fix it? @reele
   
   Yeah, but I believe that the solution to this problem still needs further 
discussion. This issue in version `3.2.0` has been carried over from version 
`1.x.x` and may have already been exploited by many people using the logic of 
`1.b`, which is why someone submitted a PR to directly bypass the 
`ProcessInstance.scheduleTime` check for `TaskInstance.startTime`.
   
   Similar issues have occurred before between versions `2.0.5` and `2.0.6`, 
and after `2.0.6`, it was also a direct bypass of the `scheduleTime` check for 
`startTime`. I have also submitted a corresponding PR, but it was directly 
rejected. Reference: #11725 and #11677. It is clear that there is a direct 
conflict of requirements regarding this issue among people.
   


-- 
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.

To unsubscribe, e-mail: [email protected]

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

Reply via email to