[GitHub] wonook commented on a change in pull request #28: [NEMO-12] Frontend support for Scala Spark

2018-06-07 Thread GitBox
wonook commented on a change in pull request #28: [NEMO-12] Frontend support for Scala Spark URL: https://github.com/apache/incubator-nemo/pull/28#discussion_r193687389 ## File path: compiler/frontend/spark/src/main/java/edu/snu/nemo/compiler/frontend/spark/sql/Dataset.java ##

[GitHub] jeongyooneo opened a new pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-10 Thread GitBox
jeongyooneo opened a new pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30 JIRA: [NEMO-63: On-demand dynamic optimization metric aggregation](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-63)

[GitHub] sanha commented on issue #28: [NEMO-12] Frontend support for Scala Spark

2018-06-11 Thread GitBox
sanha commented on issue #28: [NEMO-12] Frontend support for Scala Spark URL: https://github.com/apache/incubator-nemo/pull/28#issuecomment-396457546 @wonook Thanks for the review! I've addressed them. This is an automated mes

[GitHub] wonook closed pull request #28: [NEMO-12] Frontend support for Scala Spark

2018-06-11 Thread GitBox
wonook closed pull request #28: [NEMO-12] Frontend support for Scala Spark URL: https://github.com/apache/incubator-nemo/pull/28 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

[GitHub] sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194611550 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.ja

[GitHub] sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194611505 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.ja

[GitHub] sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194611369 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.ja

[GitHub] sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194612120 ## File path: examples/beam/src/main/java/edu/snu/nemo/examples/beam/MapReduceSkew.java

[GitHub] sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
sanha commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194611545 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.ja

[GitHub] wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194622417 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.j

[GitHub] wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194622133 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.j

[GitHub] wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-11 Thread GitBox
wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194621673 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.j

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194646153 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194646153 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194657946 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194657946 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194662468 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on issue #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
jeongyooneo commented on issue #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#issuecomment-396532748 @sanha @wonook Thank you for the review! I've addressed your comments. --

[GitHub] wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194724365 ## File path: examples/resources/sample_input_wordcount ## @@ -1,11 +1,14 @@ -banana -s

[GitHub] wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-12 Thread GitBox
wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r194723804 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.j

[GitHub] johnyangk opened a new pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-13 Thread GitBox
johnyangk opened a new pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31 JIRA: [NEMO-97: Refactor TaskExecutor and fix a sideinput bug](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-97)

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-13 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r195082501 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-13 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r195082501 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVer

[GitHub] jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-13 Thread GitBox
jeongyooneo commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r195083980 ## File path: examples/resources/sample_input_wordcount ## @@ -1,11 +1,14 @@ -bana

[GitHub] wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-13 Thread GitBox
wonook commented on a change in pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#discussion_r195299727 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/MetricCollectionBarrierVertex.j

[GitHub] jeongyooneo commented on issue #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-13 Thread GitBox
jeongyooneo commented on issue #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30#issuecomment-397175848 @wonook @sanha Thanks! I've addressed the comments.

[GitHub] sanha opened a new pull request #32: [NEMO-101] Make Coder as an execution property

2018-06-13 Thread GitBox
sanha opened a new pull request #32: [NEMO-101] Make Coder as an execution property URL: https://github.com/apache/incubator-nemo/pull/32 JIRA: [NEMO-101: Make Coder as an execution property](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-101) **Major changes:** - Make

[GitHub] wonook closed pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation

2018-06-13 Thread GitBox
wonook closed pull request #30: [NEMO-63] On-demand dynamic optimization metric aggregation URL: https://github.com/apache/incubator-nemo/pull/30 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] seojangho commented on issue #32: [NEMO-101] Make Coder as an execution property

2018-06-14 Thread GitBox
seojangho commented on issue #32: [NEMO-101] Make Coder as an execution property URL: https://github.com/apache/incubator-nemo/pull/32#issuecomment-397193089 We may have to update visualizer. @sanha May I add commits to this branch so that we can keep `json2dot.py` up-to-date with the modie

[GitHub] seojangho commented on issue #32: [NEMO-101] Make Coder as an execution property

2018-06-14 Thread GitBox
seojangho commented on issue #32: [NEMO-101] Make Coder as an execution property URL: https://github.com/apache/incubator-nemo/pull/32#issuecomment-397193089 We have to update the visualizer. @sanha May I add commits to this branch so that we can keep `json2dot.py` up-to-date with the modie

[GitHub] seojangho opened a new pull request #33: [NEMO-111] Remove ExecutionProperty.Key

2018-06-14 Thread GitBox
seojangho opened a new pull request #33: [NEMO-111] Remove ExecutionProperty.Key URL: https://github.com/apache/incubator-nemo/pull/33 JIRA: [NEMO-111: Remove enum ExecutionProperty.Key](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-111) **Major changes:** - Replaced en

[GitHub] seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31#discussion_r195323052 ## File path: compiler/frontend/spark/src/main/java/edu/snu/nemo/compiler/frontend/spark/sour

[GitHub] seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31#discussion_r195328777 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/ParentTaskDat

[GitHub] seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31#discussion_r195324465 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/ParentTaskDat

[GitHub] seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
seojangho commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31#discussion_r195328328 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/ParentTaskDat

[GitHub] johnyangk commented on issue #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
johnyangk commented on issue #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31#issuecomment-397207611 Thanks @seojangho. I've addressed your comments. Th

[GitHub] johnyangk commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
johnyangk commented on a change in pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31#discussion_r195332957 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/ParentTaskDat

[GitHub] seojangho closed pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug

2018-06-14 Thread GitBox
seojangho closed pull request #31: [NEMO-97] Refactor TaskExecutor and fix a sideinput bug URL: https://github.com/apache/incubator-nemo/pull/31 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

[GitHub] johnyangk closed pull request #32: [NEMO-101] Make Coder as an execution property

2018-06-14 Thread GitBox
johnyangk closed pull request #32: [NEMO-101] Make Coder as an execution property URL: https://github.com/apache/incubator-nemo/pull/32 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

[GitHub] wynot12 opened a new pull request #34: [NEMO-105] Move /tests/common into /common/test

2018-06-14 Thread GitBox
wynot12 opened a new pull request #34: [NEMO-105] Move /tests/common into /common/test URL: https://github.com/apache/incubator-nemo/pull/34 JIRA: [NEMO-105: Move /tests/common into /common/test] (https://issues.apache.org/jira/browse/NEMO-105) **Major changes:** - This PR moves

[GitHub] taegeonum opened a new pull request #35: [NEMO-35] Controlling logging levels while running tests

2018-06-14 Thread GitBox
taegeonum opened a new pull request #35: [NEMO-35] Controlling logging levels while running tests URL: https://github.com/apache/incubator-nemo/pull/35 JIRA: [NEMO-35: Controlling logging levels while running tests](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-35) **Majo

[GitHub] wonook closed pull request #34: [NEMO-105] Move /tests/common into /common/test

2018-06-14 Thread GitBox
wonook closed pull request #34: [NEMO-105] Move /tests/common into /common/test URL: https://github.com/apache/incubator-nemo/pull/34 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 fo

[GitHub] jeongyooneo closed pull request #35: [NEMO-35] Controlling logging levels while running tests

2018-06-14 Thread GitBox
jeongyooneo closed pull request #35: [NEMO-35] Controlling logging levels while running tests URL: https://github.com/apache/incubator-nemo/pull/35 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:

[GitHub] sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key

2018-06-14 Thread GitBox
sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key URL: https://github.com/apache/incubator-nemo/pull/33#discussion_r195634341 ## File path: compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletime/reshaping/LoopExtract

[GitHub] sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key

2018-06-14 Thread GitBox
sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key URL: https://github.com/apache/incubator-nemo/pull/33#discussion_r195634231 ## File path: compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletime/annotating/ScheduleGr

[GitHub] sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key

2018-06-14 Thread GitBox
sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key URL: https://github.com/apache/incubator-nemo/pull/33#discussion_r195632008 ## File path: common/src/main/java/edu/snu/nemo/common/ir/executionproperty/ExecutionPropertyMap.java ## @@

[GitHub] sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key

2018-06-14 Thread GitBox
sanha commented on a change in pull request #33: [NEMO-111] Remove ExecutionProperty.Key URL: https://github.com/apache/incubator-nemo/pull/33#discussion_r195634791 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/data/SerializerManager.java ## @

[GitHub] taegeonum opened a new pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
taegeonum opened a new pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36 JIRA: [NEMO-94: TITLE](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-94) **Major changes:** - Remove TODOs of resolved issues - Re-reference

[GitHub] taegeonum opened a new pull request #37: [NEMO-114] Update copyright year for 2018

2018-06-14 Thread GitBox
taegeonum opened a new pull request #37: [NEMO-114] Update copyright year for 2018 URL: https://github.com/apache/incubator-nemo/pull/37 JIRA: [NEMO-114: Update copyright year for 2018](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-114) **Major changes:** - Update copy

[GitHub] johnyangk opened a new pull request #38: [NEMO-48] Do not merge broadcasted data into a single partition

2018-06-14 Thread GitBox
johnyangk opened a new pull request #38: [NEMO-48] Do not merge broadcasted data into a single partition URL: https://github.com/apache/incubator-nemo/pull/38 JIRA: [NEMO-48: Do not merge broadcasted data into a single partition](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-48)

[GitHub] ejjeong opened a new pull request #39: [NEMO-112] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common

2018-06-14 Thread GitBox
ejjeong opened a new pull request #39: [NEMO-112] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common URL: https://github.com/apache/incubator-nemo/pull/39 JIRA: [NEMO-112: SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common](https://issues.apache.org/jira/projects/NEMO/iss

[GitHub] ejjeong commented on issue #39: [NEMO-112] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common

2018-06-14 Thread GitBox
ejjeong commented on issue #39: [NEMO-112] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common URL: https://github.com/apache/incubator-nemo/pull/39#issuecomment-397523647 ![image](https://user-images.githubusercontent.com/10492346/41452794-580af724-70ae-11e8-867a-8aa6fc6fe2b3.png)

[GitHub] jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195639503 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ## @@ -291,7 +291,7

[GitHub] jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195639462 ## File path: runtime/common/src/main/proto/ControlMessage.proto ## @@ -81,7 +81,7 @@ message BlockStateChange

[GitHub] jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195638863 ## File path: compiler/frontend/spark/src/main/java/edu/snu/nemo/compiler/frontend/spark/transform/ReduceTransform.java

[GitHub] jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195640712 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ## @@ -309,8 +309,8

[GitHub] jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
jeongyooneo commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195639572 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ## @@ -309,8 +309,8

[GitHub] sanha commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-14 Thread GitBox
sanha commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195640944 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ## @@ -379,7 +378,6 @@ pri

[GitHub] yunseong opened a new pull request #40: Rename RoundRobinSchedulingPolicy to MinOccupancyFirstSchedulingPolicy

2018-06-14 Thread GitBox
yunseong opened a new pull request #40: Rename RoundRobinSchedulingPolicy to MinOccupancyFirstSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/40 JIRA: [NEMO-74: Rename RoundRobinSchedulingPolicy](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-74) **Majo

[GitHub] ejjeong opened a new pull request #41: [Nemo 108] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Master

2018-06-14 Thread GitBox
ejjeong opened a new pull request #41: [Nemo 108] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Master URL: https://github.com/apache/incubator-nemo/pull/41 JIRA: [NEMO-108: SonarCloud Bugs and Vulnerabilities for Nemo Runtime Master](https://issues.apache.org/jira/projects/NEMO/iss

[GitHub] johnyangk closed pull request #40: [NEMO-74] Rename RoundRobinSchedulingPolicy to MinOccupancyFirstSchedulingPolicy

2018-06-14 Thread GitBox
johnyangk closed pull request #40: [NEMO-74] Rename RoundRobinSchedulingPolicy to MinOccupancyFirstSchedulingPolicy URL: https://github.com/apache/incubator-nemo/pull/40 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] DifferentSC opened a new pull request #42: [NEMO-41] SonarCloud Bugs and Vulnerabilities for NemoCommon

2018-06-14 Thread GitBox
DifferentSC opened a new pull request #42: [NEMO-41] SonarCloud Bugs and Vulnerabilities for NemoCommon URL: https://github.com/apache/incubator-nemo/pull/42 JIRA: [NEMO-41: SonarCloud Bugs and Vulnerabilities for NemoCommon](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-41)

[GitHub] ejjeong opened a new pull request #43: [NEMO-115] SonarCloud Bugs and Vulnerabilities for Nemo Client

2018-06-14 Thread GitBox
ejjeong opened a new pull request #43: [NEMO-115] SonarCloud Bugs and Vulnerabilities for Nemo Client URL: https://github.com/apache/incubator-nemo/pull/43 JIRA: [NEMO-115: SonarCloud Bugs and Vulnerabilities for Nemo Client](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-115)

[GitHub] sanha closed pull request #41: [Nemo 108] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Master

2018-06-14 Thread GitBox
sanha closed pull request #41: [Nemo 108] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Master URL: https://github.com/apache/incubator-nemo/pull/41 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 prove

[GitHub] DifferentSC opened a new pull request #44: [NEMO-113] Update README

2018-06-14 Thread GitBox
DifferentSC opened a new pull request #44: [NEMO-113] Update README URL: https://github.com/apache/incubator-nemo/pull/44 JIRA: [NEMO-113: TITLE](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-113) **Major changes:** - Fix broken links in README.md **Minor changes

[GitHub] jeongyooneo closed pull request #38: [NEMO-48] Do not merge broadcasted data into a single partition

2018-06-15 Thread GitBox
jeongyooneo closed pull request #38: [NEMO-48] Do not merge broadcasted data into a single partition URL: https://github.com/apache/incubator-nemo/pull/38 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 proven

[GitHub] sanha closed pull request #43: [NEMO-115] SonarCloud Bugs and Vulnerabilities for Nemo Client

2018-06-15 Thread GitBox
sanha closed pull request #43: [NEMO-115] SonarCloud Bugs and Vulnerabilities for Nemo Client URL: https://github.com/apache/incubator-nemo/pull/43 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:

[GitHub] seojangho opened a new pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-15 Thread GitBox
seojangho opened a new pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45 JIRA: [NEMO-103: Communication between Driver and Client](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-103) **Major changes:*

[GitHub] sanha closed pull request #39: [NEMO-112] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common

2018-06-15 Thread GitBox
sanha closed pull request #39: [NEMO-112] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Common URL: https://github.com/apache/incubator-nemo/pull/39 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 prove

[GitHub] ejjeong opened a new pull request #46: SonarCloud Bugs and Vulnerabilities for Nemo Runtime Executor

2018-06-15 Thread GitBox
ejjeong opened a new pull request #46: SonarCloud Bugs and Vulnerabilities for Nemo Runtime Executor URL: https://github.com/apache/incubator-nemo/pull/46 JIRA: [NEMO-107: SonarCloud Bugs and Vulnerabilities for Nemo Runtime Executor](https://issues.apache.org/jira/projects/NEMO/issues/NEM

[GitHub] sanha closed pull request #46: [NEMO-107] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Executor

2018-06-15 Thread GitBox
sanha closed pull request #46: [NEMO-107] SonarCloud Bugs and Vulnerabilities for Nemo Runtime Executor URL: https://github.com/apache/incubator-nemo/pull/46 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 pro

[GitHub] taegeonum commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-15 Thread GitBox
taegeonum commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195668126 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ## @@ -379,7 +378,6 @

[GitHub] taegeonum commented on issue #36: [NEMO-94] Clean up legacy TODOs

2018-06-15 Thread GitBox
taegeonum commented on issue #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#issuecomment-397555075 @sanha @jeongyooneo I've addressed your comments. Thanks! This is an automated mess

[GitHub] sanha commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-15 Thread GitBox
sanha commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195671522 ## File path: compiler/frontend/spark/src/main/java/edu/snu/nemo/compiler/frontend/spark/transform/ReduceTransform.java #

[GitHub] sanha commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-15 Thread GitBox
sanha commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195671720 ## File path: compiler/frontend/spark/src/main/java/edu/snu/nemo/compiler/frontend/spark/transform/ReduceTransform.java #

[GitHub] yunseong opened a new pull request #47: Nemo 83

2018-06-15 Thread GitBox
yunseong opened a new pull request #47: Nemo 83 URL: https://github.com/apache/incubator-nemo/pull/47 JIRA: [NEMO-83: Move /tests/runtime into /runtime/tests](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-83) **Major changes:** - This PR moves the tests/runtime to nemo-

[GitHub] taegeonum commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-15 Thread GitBox
taegeonum commented on a change in pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36#discussion_r195689695 ## File path: compiler/frontend/spark/src/main/java/edu/snu/nemo/compiler/frontend/spark/transform/ReduceTransform.java

[GitHub] johnyangk closed pull request #47: [NEMO-83] Move /tests/runtime into /runtime/tests

2018-06-15 Thread GitBox
johnyangk closed pull request #47: [NEMO-83] Move /tests/runtime into /runtime/tests URL: https://github.com/apache/incubator-nemo/pull/47 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 i

[GitHub] jeongyooneo closed pull request #36: [NEMO-94] Clean up legacy TODOs

2018-06-15 Thread GitBox
jeongyooneo closed pull request #36: [NEMO-94] Clean up legacy TODOs URL: https://github.com/apache/incubator-nemo/pull/36 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] jeongyooneo closed pull request #44: [NEMO-113] Update README

2018-06-15 Thread GitBox
jeongyooneo closed pull request #44: [NEMO-113] Update README URL: https://github.com/apache/incubator-nemo/pull/44 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] jeongyooneo closed pull request #42: [NEMO-41] SonarCloud Bugs and Vulnerabilities for NemoCommon

2018-06-15 Thread GitBox
jeongyooneo closed pull request #42: [NEMO-41] SonarCloud Bugs and Vulnerabilities for NemoCommon URL: https://github.com/apache/incubator-nemo/pull/42 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 provenanc

[GitHub] seojangho commented on issue #33: [NEMO-111] Remove ExecutionProperty.Key

2018-06-17 Thread GitBox
seojangho commented on issue #33: [NEMO-111] Remove ExecutionProperty.Key URL: https://github.com/apache/incubator-nemo/pull/33#issuecomment-397936829 @sanha Thanks for the comment. Please check the updates. This is an automat

[GitHub] seojangho commented on issue #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-17 Thread GitBox
seojangho commented on issue #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#issuecomment-397954545 Okay. I've managed to address Spark-related test failures. @wonook @sanha Please have a look. -

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195985453 ## File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java ## @@ -135,15 +13

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195988229 ## File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java ## @@ -102,13 +115,10 @@

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195987180 ## File path: client/src/main/java/edu/snu/nemo/client/DriverRPCServer.java ## @@ -0,0 +1,182 @@

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195986770 ## File path: runtime/driver/src/main/java/edu/snu/nemo/driver/ClientRPC.java ## @@ -0,0 +1,147

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195987538 ## File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java ## @@ -72,6 +74,15 @@ pr

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195986628 ## File path: runtime/driver/src/main/java/edu/snu/nemo/driver/ClientRPC.java ## @@ -0,0 +1,147

[GitHub] wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver

2018-06-18 Thread GitBox
wonook commented on a change in pull request #45: [NEMO-103] Implement RPC between Client and Driver URL: https://github.com/apache/incubator-nemo/pull/45#discussion_r195986212 ## File path: runtime/driver/src/main/java/edu/snu/nemo/driver/ClientRPC.java ## @@ -0,0 +1,147

[GitHub] sanha opened a new pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
sanha opened a new pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48 JIRA: [NEMO-72: Instance-based Encoder/Decoder interface](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-72) **Major changes:**

[GitHub] johnyangk opened a new pull request #49: [NEMO-87] Remove unused BlockStates

2018-06-18 Thread GitBox
johnyangk opened a new pull request #49: [NEMO-87] Remove unused BlockStates URL: https://github.com/apache/incubator-nemo/pull/49 JIRA: [NEMO-87: Remove unused BlockStates](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-87) **Major changes:** - READY, LOST_BEFORE_COMMIT

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196273838 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/NemoPipelineVis

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196274635 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/coder/BeamEncod

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196273958 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/coder/BeamDecod

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196273415 ## File path: common/src/main/java/edu/snu/nemo/common/coder/IntEncoder.java ## @@ -0,0 +1,66

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196272799 ## File path: common/src/main/java/edu/snu/nemo/common/coder/IntDecoder.java ## @@ -0,0 +1,69

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196272541 ## File path: common/src/main/java/edu/snu/nemo/common/coder/Decoder.java ## @@ -0,0 +1,89 @@

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196275302 ## File path: runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/data/SerializerManager.java

[GitHub] seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

2018-06-18 Thread GitBox
seojangho commented on a change in pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196275597 ## File path: common/src/main/java/edu/snu/nemo/common/coder/IntDecoder.java ## @@ -0,0 +1,69

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