[GitHub] seojangho opened a new pull request #77: [NEMO-161] Make all injectable constructors private

seojangho opened a new pull request #77: [NEMO-161] Make all injectable constructors private URL: https://github.com/apache/incubator-nemo/pull/77 JIRA: [NEMO-161: Make all injectable constructors private](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-161) **Major

[GitHub] shpark opened a new pull request #75: [NEMO-151] Add OutputWriters for additional tagged outputs

shpark opened a new pull request #75: [NEMO-151] Add OutputWriters for additional tagged outputs URL: https://github.com/apache/incubator-nemo/pull/75 JIRA: [NEMO-151: Add OutputWriters for addtional tagged outputs](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-151)

[GitHub] seojangho closed pull request #71: [NEMO-55] Handle NCS Master-to-Executor RPC failures

seojangho closed pull request #71: [NEMO-55] Handle NCS Master-to-Executor RPC failures URL: https://github.com/apache/incubator-nemo/pull/71 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

[GitHub] wonook opened a new pull request #73: [Nemo-62] Support Multiple Jobs Submission in a Single User Program

wonook opened a new pull request #73: [Nemo-62] Support Multiple Jobs Submission in a Single User Program URL: https://github.com/apache/incubator-nemo/pull/73 JIRA: [NEMO-62: Support Multiple Jobs Submission in a Single User

[GitHub] taegeonum opened a new pull request #120: [NEMO-217] Immediately emits elements in FlattenTransform

taegeonum opened a new pull request #120: [NEMO-217] Immediately emits elements in FlattenTransform URL: https://github.com/apache/incubator-nemo/pull/120 JIRA: [NEMO-217: Immediately emits elements in FlattenTransform](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-217)

[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_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_r172710971 ## 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_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] 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_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_r172758202 ## File path:

[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] 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] 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_r172759613 ## File path: tests/src/test/java/edu/snu/nemo/tests/runtime/RuntimeTestUtil.java ## @@

[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

[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_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_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_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_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_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_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 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_r173046431 ## File path:

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

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

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

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

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

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

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

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

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

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

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

sanha commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#discussion_r173049096 ## File path: examples/beam/src/main/java/edu/snu/nemo/examples/beam/AlternatingLeastSquare.java ## @@ -270,7

[GitHub] sanha commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining

sanha commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#issuecomment-371376883 For the @johnyangk's comment (avoiding hashing and creating objects), I'd suggest the next model. - Build a DAG of `TaskWrapper` (or something

[GitHub] sanha commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining

sanha commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#issuecomment-371376883 For the @johnyangk's comment (avoiding hashing and creating objects), I'd suggest the following model. - Build a DAG of `TaskWrapper` (or

[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_r173067005 ## 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_r173078453 ## 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_r173065772 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/scheduler/SchedulingPolicy.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_r173069497 ## File path: tests/src/test/java/edu/snu/nemo/tests/runtime/RuntimeTestUtil.java ## @@

[GitHub] sanha commented on issue #5: [NEMO-27] Element Wise Block Write

sanha commented on issue #5: [NEMO-27] Element Wise Block Write URL: https://github.com/apache/incubator-nemo/pull/5#issuecomment-372105859 I've updated this pr and it's description. Please have a look @jeongyooneo! This is

[GitHub] wonook closed pull request #6: [NEMO-34] SonarCloud

wonook closed pull request #6: [NEMO-34] SonarCloud URL: https://github.com/apache/incubator-nemo/pull/6 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 (from a

[GitHub] seojangho closed pull request #7: [NEMO-34] SonarCloud

seojangho closed pull request #7: [NEMO-34] SonarCloud URL: https://github.com/apache/incubator-nemo/pull/7 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] wonook opened a new pull request #7: [NEMO-34] SonarCloud

wonook opened a new pull request #7: [NEMO-34] SonarCloud URL: https://github.com/apache/incubator-nemo/pull/7 JIRA: [NEMO-34](https://issues.apache.org/jira/browse/NEMO-34) **Major changes:** - Edit Travis.yml to support sonarcloud at

[GitHub] jeongyooneo commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining

jeongyooneo commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#issuecomment-373625012 Thanks for the review! I've addressed most of them, except the following two: 1) For refactoring metric collection code into a separate

[GitHub] jeongyooneo commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining

jeongyooneo commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#issuecomment-371838721 @bgchun I?ve modified the ?TaskGroupExecutor? to element-wise version and checked that all ITCases worked correctly with this PR. If you

[GitHub] jeongyooneo commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining

jeongyooneo commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#issuecomment-371838721 @bgchun I?ve modified the TaskGroupExecutorTest to element-wise version and checked that all ITCases worked correctly with this PR. If you

[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] jeongyooneo commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

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

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

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

[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 #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] jeongyooneo commented on a change in pull request #2: [NEMO-7] Intra-TaskGroup pipelining

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

[GitHub] sanha opened a new pull request #5: [NEMO-27] Element Wise Block Write

sanha opened a new pull request #5: [NEMO-27] Element Wise Block Write URL: https://github.com/apache/incubator-nemo/pull/5 JIRA: [NEMO-27: Element Wise Block Write](https://issues.apache.org/jira/browse/NEMO-27) **Major changes:** - Changed interface of `Partitioner` to accept

[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] seojangho opened a new pull request #14: [NEMO-38] Fix bugs and vulnerabilities for RuntimeMaster and RuntimeCommon

seojangho opened a new pull request #14: [NEMO-38] Fix bugs and vulnerabilities for RuntimeMaster and RuntimeCommon URL: https://github.com/apache/incubator-nemo/pull/14 JIRA: [NEMO-38: SonarCloud Bugs and Vulnerabilities for RuntimeMaster and

[GitHub] wonook closed pull request #14: [NEMO-38] Fix bugs and vulnerabilities for RuntimeMaster and RuntimeCommon

wonook closed pull request #14: [NEMO-38] Fix bugs and vulnerabilities for RuntimeMaster and RuntimeCommon URL: https://github.com/apache/incubator-nemo/pull/14 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

[GitHub] wonook commented on issue #14: [NEMO-38] Fix bugs and vulnerabilities for RuntimeMaster and RuntimeCommon

wonook commented on issue #14: [NEMO-38] Fix bugs and vulnerabilities for RuntimeMaster and RuntimeCommon URL: https://github.com/apache/incubator-nemo/pull/14#issuecomment-378811246 Looks simple. I'll test and merge This is

[GitHub] seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#discussion_r177318913 ## File path:

[GitHub] seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#discussion_r177312494 ## File path:

[GitHub] seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#discussion_r177311783 ## File path:

[GitHub] seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#discussion_r177312843 ## File path:

[GitHub] seojangho commented on issue #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on issue #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#issuecomment-376410720 @sanha Thanks for the review! Please check the updates.

[GitHub] seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on a change in pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#discussion_r177311783 ## File path:

[GitHub] seojangho commented on issue #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on issue #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#issuecomment-376775124 We may not want to merge this branch to `master` before ASF Jenkins is ready for CI,

[GitHub] seojangho commented on issue #10: [NEMO-31] Community Standards

seojangho commented on issue #10: [NEMO-31] Community Standards URL: https://github.com/apache/incubator-nemo/pull/10#issuecomment-376774728 Let us go forward. @jooykim Please check the updates and merge this PR if you can approve it. Thanks.

[GitHub] seojangho commented on issue #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

seojangho commented on issue #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11#issuecomment-376775124 We may not want to merge this branch to `master` before ASF Jenkins is ready for CI,

[GitHub] sanha closed pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling

sanha closed pull request #11: [NEMO-42] Make SchedulerRunner reactive, Ensure reverse-topological ordering in scheduling URL: https://github.com/apache/incubator-nemo/pull/11 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] jooykim closed pull request #10: [NEMO-31] Community Standards

jooykim closed pull request #10: [NEMO-31] Community Standards URL: https://github.com/apache/incubator-nemo/pull/10 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

[GitHub] wonook opened a new pull request #13: [NEMO-43] Sonar cloud script

wonook opened a new pull request #13: [NEMO-43] Sonar cloud script URL: https://github.com/apache/incubator-nemo/pull/13 JIRA: [NEMO-43: Sonar cloud script](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-43) **Major changes:** - Add sonar qube script to run sonar

[GitHub] johnyangk commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining

johnyangk commented on issue #2: [NEMO-7] Intra-TaskGroup pipelining URL: https://github.com/apache/incubator-nemo/pull/2#issuecomment-373991169 Thanks @sanha. I'll make another pass once @jeongyooneo addresses the comments here.

[GitHub] wonook opened a new pull request #9: [NEMO-40] Solve SonarCloud issues for Frontend and Examples

wonook opened a new pull request #9: [NEMO-40] Solve SonarCloud issues for Frontend and Examples URL: https://github.com/apache/incubator-nemo/pull/9 JIRA: [NEMO-40](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-40) **Major changes:** - Fixes travis script to build

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

<    2   3   4   5   6   7   8   9   10   11   >