jeongyooneo commented on a change in pull request #20: [NEMO-76] Rename
TaskGroup to Task
URL: https://github.com/apache/incubator-nemo/pull/20#discussion_r189456094
##########
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/JobStateManager.java
##########
@@ -260,39 +260,38 @@ public synchronized void onStageStateChanged(final
String stageId, final StageSt
}
/**
- * Updates the state of a task group.
- * Task group state changes can occur both in master and executor.
+ * Updates the state of a task.
+ * Task state changes can occur both in master and executor.
* State changes that occur in master are
* initiated in {@link
edu.snu.nemo.runtime.master.scheduler.BatchSingleJobScheduler}.
* State changes that occur in executors are sent to master as a control
message,
* and the call to this method is initiated in {@link
edu.snu.nemo.runtime.master.scheduler.BatchSingleJobScheduler}
* when the message/event is received.
- * A task group completion implies completion of all its tasks.
+ * A task completion implies completion of all its tasks.
Review comment:
Maybe remove this line?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
