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
##
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)
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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.
--
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
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
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)
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
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
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
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
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.
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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:
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
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
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
##
@@
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
##
@
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
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
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)
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
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)
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
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
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
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
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
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
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
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
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
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)
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)
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
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
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
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:
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:*
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
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
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
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 @
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
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
#
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
#
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-
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
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
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
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
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
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
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.
-
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
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 @@
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 @@
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
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
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
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
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:**
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
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
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
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
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
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
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 @@
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
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
401 - 500 of 1748 matches
Mail list logo