Hisoka-X commented on code in PR #9749:
URL: https://github.com/apache/seatunnel/pull/9749#discussion_r2319157107
##########
seatunnel-core/seatunnel-starter/src/main/java/org/apache/seatunnel/core/starter/seatunnel/command/ClientExecuteCommand.java:
##########
@@ -211,8 +211,11 @@ public void execute() throws CommandExecuteException {
0,
TimeUnit.SECONDS);
}
-
- // wait for job complete
+ // In the two scenarios of pending state cancellation and task
resource insufficient
+ // error, when the corresponding JobStatus is not stored, the
+ // CoordinatorService.getJobStatus method will return NULL,
which can be avoided by
+ // sleeping for 200 milliseconds
+ TimeUnit.MILLISECONDS.sleep(200);
Review Comment:
-1. We should NEVER use sleep to make sure the status change. It's very
unstable!
--
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]
