sanha commented on a change in pull request #115: [NEMO-96] Modularize
DataSkewPolicy to use MetricVertex and BarrierVertex
URL: https://github.com/apache/incubator-nemo/pull/115#discussion_r214383751
##########
File path:
compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletime/composite/SkewCompositePass.java
##########
@@ -22,8 +22,6 @@
/**
* Pass to modify the DAG for a job to perform data skew.
- * It adds a {@link
edu.snu.nemo.common.ir.vertex.MetricCollectionBarrierVertex} before Shuffle
edges,
- * to make a barrier before it, and to use the metrics to repartition the
skewed data.
* NOTE: we currently put the SkewCompositePass at the end of the list for
each policies, as it needs to take a
Review comment:
Please update the description (rather than just removing it).
----------------------------------------------------------------
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
