johnyangk commented on a change in pull request #24: [NEMO-79] Clean up the
legacy Task
URL: https://github.com/apache/incubator-nemo/pull/24#discussion_r191732056
##########
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/RuntimeIdGenerator.java
##########
@@ -76,36 +79,25 @@ public static String generateStageId(final Integer
stageId) {
}
/**
- * Generates the ID for {@link
edu.snu.nemo.runtime.common.plan.physical.Task}.
- *
- * @param irVertexId the ID of the IR vertex.
- * @return the generated ID
- */
- public static String generateLogicalTaskId(final String irVertexId) {
- return "Task-" + irVertexId;
- }
-
- /**
- * Generates the ID for {@link
edu.snu.nemo.runtime.common.plan.physical.Task}.
+ * Generates the ID for the physical representation of an IRVertex.
*
- * @param index the index of the physical task.
- * @param logicalTaskId the logical ID of the task.
+ * @param irVertexId the id of the IRVertex.
+ * @param index the index of the physical task.
* @return the generated ID
*/
- public static String generatePhysicalTaskId(final int index,
- final String logicalTaskId) {
- return logicalTaskId + PHYSICAL_TASK_ID_SPLITTER + index;
+ public static String generatePhysicalIRVertexId(final String irVertexId,
Review comment:
Thanks! I've missed those. I've removed the related methods and variables,
as they don't have a functional purpose.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
