You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@nemo.apache.org by GitBox <gi...@apache.org> on 2018/11/06 05:57:39 UTC

[GitHub] johnyangk commented on issue #150: [NEMO-269] Direct translation from Beam DAG to Nemo DAG

johnyangk commented on issue #150: [NEMO-269] Direct translation from Beam DAG to Nemo DAG
URL: https://github.com/apache/incubator-nemo/pull/150#issuecomment-436139914
 
 
   @taegeonum I've addressed your comment on the method name.
   Regarding the tests, they are disabled because this PR disables the previous partial combining optimization and thus changes the number of vertices in the DAGs used in the tests.
   
   Please see:
   https://github.com/apache/incubator-nemo/pull/150/files#diff-5b36549ea7bd9b8064024cfd8db046a8R282

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services