EricJoy2048 commented on code in PR #5450:
URL: https://github.com/apache/seatunnel/pull/5450#discussion_r1326828380


##########
seatunnel-engine/seatunnel-engine-core/src/main/java/org/apache/seatunnel/engine/core/job/JobStatus.java:
##########
@@ -29,10 +29,10 @@ public enum JobStatus {
     /** Job is newly created, no task has started to run. */
     CREATED(EndState.NOT_END),
 
-    /** Job is begin schedule but some task not deploy complete. */
+    /** Job will scheduler every pipeline */
     SCHEDULED(EndState.NOT_END),
 
-    /** Some tasks are scheduled or running, some may be pending, some may be 
finished. */
+    /** Job is running and begine to scheduler pipeline running. */

Review Comment:
   ```suggestion
       /** The job is already running, and each pipeline is already running. */
   ```



##########
seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/CoordinatorService.java:
##########
@@ -759,9 +703,6 @@ public void printJobDetailInfo() {
                                         case CREATED:
                                             createdJobCount.addAndGet(1);
                                             break;
-                                        case SCHEDULED:

Review Comment:
   `SCHEDULED` job need to keep it because the status of the job can be 
SCHEDULED.



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