ashulin commented on code in PR #2556:
URL:
https://github.com/apache/incubator-seatunnel/pull/2556#discussion_r957964856
##########
seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/checkpoint/operation/TaskCompletedOperation.java:
##########
@@ -54,12 +59,10 @@ protected void readInternal(ObjectDataInput in) throws
IOException {
}
@Override
- protected PassiveCompletableFuture<?> doRun() throws Exception {
- CheckpointCoordinator checkpointCoordinator = ((SeaTunnelServer)
getService())
+ public void run() {
Review Comment:
`PhysicalVertex.taskFuture` is a notification of task group granularity,
which does not meet the requirements of checkpoint.
--
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]