polarcoke2 commented on a change in pull request #292:
URL: https://github.com/apache/incubator-nemo/pull/292#discussion_r449367330
##########
File path: common/src/main/java/org/apache/nemo/common/ir/IRDAG.java
##########
@@ -650,6 +876,112 @@ private IREdge edgeToMessageAggregator(final IRVertex
trigger,
return newEdge;
}
+ // These similar four helper methods are for inserting TaskSizeSplitterVertex
Review comment:
Makes sense to me either. I refactored the code as you mentioned. Thanks!
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
