ruanwenjun commented on PR #15358: URL: https://github.com/apache/dolphinscheduler/pull/15358#issuecomment-1868753622
> `TODO: We can put this logic in task plugin, but this will cause all task plugin need to handle this situation.` What does this TODO mean? Doing it in `WorkerTaskExecutor.java` seems better than in task plugins. Some task instance may store some info in its working, and when we failover or recover(Flink) we may need to read this data. But right now, we didn't have this kind of task plugin. -- 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]
