[GitHub] seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172762612 ## File path:

[GitHub] seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172762043 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/transform/Transform.java ## @@ -32,16

[GitHub] seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172762337 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/transform/FlattenTransform.java

[GitHub] seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172762740 ## File path: conf/src/main/java/edu/snu/nemo/conf/JobConf.java ## @@ -103,7 +103,7 @@ * Path to the JSON

[GitHub] seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172762472 ## File path:

[GitHub] seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

seojangho commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172762523 ## File path:

[GitHub] seojangho commented on issue #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on issue #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#issuecomment-371051613 @sanha Thanks for the review. Please check the updates.

[GitHub] seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172759613 ## File path: tests/src/test/java/edu/snu/nemo/tests/runtime/RuntimeTestUtil.java ## @@

[GitHub] seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172759560 ## File path:

[GitHub] seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172758202 ## File path:

[GitHub] seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172755632 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java

[GitHub] seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172754621 ## File path:

[GitHub] seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

seojangho commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172753133 ## File path: common/src/main/java/edu/snu/nemo/common/ir/Readable.java ## @@ -28,5

[GitHub] johnyangk commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

johnyangk commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172750797 ## File path: tests/src/test/java/edu/snu/nemo/tests/runtime/RuntimeTestUtil.java ## @@

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172728122 ## File path:

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172735237 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/scheduler/SchedulerRunner.java

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172723450 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ##

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172748732 ## File path:

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172721727 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/BlockManagerMaster.java

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172725494 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ##

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172726613 ## File path:

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172736279 ## File path:

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172720282 ## File path:

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172721711 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/BlockManagerMaster.java

[GitHub] sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on a change in pull request #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#discussion_r172736526 ## File path:

[GitHub] sanha commented on issue #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy

sanha commented on issue #1: [NEMO-26] Implement SourceLocationAwareSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/1#issuecomment-370998667 Thanks for the work! I'm reviewing this change now. By the way, could you update the JIRA issue description for the

Podling Report Reminder - March 2018

Dear podling, This email was sent by an automated system on behalf of the Apache Incubator PMC. It is an initial reminder to give you plenty of time to prepare your quarterly board report. The board meeting is scheduled for Wed, 21 March 2018, 10:30 am PDT. The report for your podling will form

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172710456 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172709998 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172709098 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172710709 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172710239 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172704416 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172706819 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172711081 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172711250 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172708316 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172707585 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172711570 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172708330 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172707978 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172710298 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r172710971 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/TaskGroupExecutor.java ## @@

[GitHub] sanha closed pull request #4: [NEMO-28] Javadoc and Travis

sanha closed pull request #4: [NEMO-28] Javadoc and Travis URL: https://github.com/apache/incubator-nemo/pull/4 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] jooykim commented on issue #4: [NEMO-28] Javadoc and Travis

jooykim commented on issue #4: [NEMO-28] Javadoc and Travis URL: https://github.com/apache/incubator-nemo/pull/4#issuecomment-370780405 @sanha Please go ahead and merge this PR if you're fine with what @wonook said. Thanks!

[GitHub] wonook commented on issue #4: [NEMO-28] Javadoc and Travis

wonook commented on issue #4: [NEMO-28] Javadoc and Travis URL: https://github.com/apache/incubator-nemo/pull/4#issuecomment-370778158 @jooykim My bad. We should set up the PR template on the repo to remove confusion. I'll add a ticket on JIRA for that.

[GitHub] jooykim commented on issue #4: [NEMO-28] Javadoc and Travis

jooykim commented on issue #4: [NEMO-28] Javadoc and Travis URL: https://github.com/apache/incubator-nemo/pull/4#issuecomment-370777667 @wonook Can we remove the "The contents below prerequisites and setup should soon move to either the website or the wiki." under other comments in the PR

[GitHub] wonook commented on a change in pull request #4: [NEMO-28] Javadoc and Travis

wonook commented on a change in pull request #4: [NEMO-28] Javadoc and Travis URL: https://github.com/apache/incubator-nemo/pull/4#discussion_r172510877 ## File path: common/src/main/java/edu/snu/nemo/common/test/ArgBuilder.java ## @@ -19,12 +19,14 @@ import

[GitHub] wonook opened a new pull request #4: [NEMO-28] Javadoc and Travis

wonook opened a new pull request #4: [NEMO-28] Javadoc and Travis URL: https://github.com/apache/incubator-nemo/pull/4 JIRA: [NEMO-28: Javadoc and Travis](https://issues.apache.org/jira/browse/NEMO-28) **Major changes:** - Travis script fixed to catch errors **Minor