johnyangk commented on a change in pull request #85: [NEMO-172] Implement one 
partition per one element partitioner
URL: https://github.com/apache/incubator-nemo/pull/85#discussion_r208069918
 
 

 ##########
 File path: 
common/src/main/java/edu/snu/nemo/common/ir/edge/executionproperty/PartitionerProperty.java
 ##########
 @@ -44,6 +44,7 @@ public static PartitionerProperty of(final Value value) {
   public enum Value {
     DataSkewHashPartitioner,
     HashPartitioner,
-    IntactPartitioner
+    IntactPartitioner,
+    IncrementPartitioner
 
 Review comment:
   I can't think of a good name, but it'd be nice if the name reflects the fact 
that each partition will have only one element. (or each element will go into a 
separate partition)

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

Reply via email to