wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201423203
##
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/JobStateManager.
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201426548
##
File path:
runtime/master/src/test/java/edu/snu/nemo/runtime/master/scheduler/TaskRe
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201409021
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/metric/StateMetr
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201408343
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201412158
##
File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201426540
##
File path:
runtime/master/src/test/java/edu/snu/nemo/runtime/master/scheduler/BatchS
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201423085
##
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/MetricStore.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201409141
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/plan/PhysicalPla
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201414667
##
File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201400646
##
File path:
examples/spark/src/main/java/edu/snu/nemo/examples/spark/JavaWordAndLineC
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r201415610
##
File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java
johnyangk closed pull request #72: [NEMO-59] Skewed data-aware executor
allocation
URL: https://github.com/apache/incubator-nemo/pull/72
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
johnyangk commented on issue #71: [NEMO-55] Handle NCS Master-to-Executor RPC
failures
URL: https://github.com/apache/incubator-nemo/pull/71#issuecomment-403994990
@seojangho I've resolved the conflict with the master
This is
wynot12 commented on issue #73: [Nemo-62] Support Multiple Jobs Submission in a
Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#issuecomment-404042300
One more request.
Multiple jobs mean multiple _pipelines_ in Beam and multiple _actions_ in
Spark, right?
sanha opened a new pull request #74: [NEMO-141] Make vertices receiving push
edge not comply executor slot
URL: https://github.com/apache/incubator-nemo/pull/74
JIRA: [NEMO-141: Make vertices receiving push edge not comply executor
slot](https://issues.apache.org/jira/projects/NEMO/issues/
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)
**Maj
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 thi
johnyangk closed pull request #74: [NEMO-141] Make vertices receiving push edge
not comply executor slot
URL: https://github.com/apache/incubator-nemo/pull/74
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 pr
shpark commented on issue #75: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/75#issuecomment-404701618
Messed up while resolving conflicts. I will send a new pull request.
--
shpark closed pull request #75: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/75
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
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202254889
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@ -
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202254948
##
File path:
common/src/main/java/edu/snu/nemo/common/ir/vertex/transform/Transform.jav
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202262396
##
File path:
examples/spark/src/main/java/edu/snu/nemo/examples/spark/JavaWordAndLineCo
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202262425
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/metric/StateTrans
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202262415
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/metric/StateMetri
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202264147
##
File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java
#
shpark opened a new pull request #76: [NEMO-151] Add OutputWriters for
additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76
JIRA: [NEMO-151: Add OutputWriters for addtional tagged
outputs](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-151)
**Maj
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202282748
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
#
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 changes
johnyangk closed pull request #77: [NEMO-161] Make all injectable constructors
private
URL: https://github.com/apache/incubator-nemo/pull/77
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
sanha commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202512383
##
File path:
runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/TaskExecutor.java
sanha commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202512438
##
File path:
runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/TaskExecutor.java
sanha commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202512453
##
File path:
runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/VertexHarness.java
shpark commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202512750
##
File path:
runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/VertexHarness.jav
sanha commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202512775
##
File path:
runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/VertexHarness.java
shpark commented on issue #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405016463
@sanha Thank you for you reviews! I will take your suggestions into account
and add commits.
-
shpark commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202512819
##
File path:
runtime/executor/src/main/java/edu/snu/nemo/runtime/executor/task/TaskExecutor.java
shpark commented on issue #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405016888
By the way, I've found out that we should've update the
AdditionalTagOutputProperty, if new vertices are inserted (for exam
shpark edited a comment on issue #76: [NEMO-151] Add OutputWriters for
additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405016888
By the way, I've found out that we should've update the
`AdditionalTagOutputProperty` of the sources vertex, if new
sanha commented on issue #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405018033
@shpark it can be a part of this pr. Please add the commit also. I'll take a
look.
shpark commented on issue #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405018110
I've add a new commit regarding update of `AddiitonalTagOutputProperty` of a
source vertex, which is a direct parent of new
shpark edited a comment on issue #76: [NEMO-151] Add OutputWriters for
additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405018110
I've add a new commit regarding update of `AddiitonalTagOutputProperty` of a
source vertex, which is a direct parent
sanha commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202515893
##
File path:
examples/beam/src/test/java/edu/snu/nemo/examples/beam/PartitionWordsByLengthITCase.
sanha commented on issue #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-405026240
@shpark I agree with your opinion. I've created an
[issue](https://issues.apache.org/jira/browse/NEMO-162) in JIRA. Thanks f
shpark commented on a change in pull request #76: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#discussion_r202529103
##
File path:
examples/beam/src/test/java/edu/snu/nemo/examples/beam/PartitionWordsByLengthITCase
wonook commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-405507821
I'll merge once you address the comment related to the `plangenerator`
package and resolve the conflicts
--
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202941942
##
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/MetricStore.java
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202942040
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/plan/PhysicalPlan
wonook commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r202942700
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@ -
seojangho opened a new pull request #78: [NEMO-163] Update README.md to use
Hadoop 2.7.2
URL: https://github.com/apache/incubator-nemo/pull/78
JIRA: [NEMO-163: Update README.md to use Hadoop
2.7.2](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-163)
**Major changes:**
-
johnyangk closed pull request #78: [NEMO-163] Update README.md to use Hadoop
2.7.2
URL: https://github.com/apache/incubator-nemo/pull/78
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
seojangho commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203346684
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@
seojangho commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203345239
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@
wynot12 commented on issue #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69#issuecomment-406014682
@wonook Thanks for review!
I've renamed `nemo-runtime-plangenerator` pacakge to `nemo-runtime-test`.
Please take a look :)
-
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203576446
##
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/MetricStore.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203573993
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203572667
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203574025
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/metric/StateMetr
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203576950
##
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.ja
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203573970
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203575766
##
File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203577549
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203573410
##
File path:
common/src/main/java/edu/snu/nemo/common/ir/vertex/transform/Transform.ja
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203574034
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/metric/StateTran
wynot12 commented on a change in pull request #73: [Nemo-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203574533
##
File path:
runtime/common/src/main/java/edu/snu/nemo/runtime/common/plan/PhysicalPla
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203606399
##
File path:
common/src/main/java/edu/snu/nemo/common/ir/vertex/transform/Transform.jav
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203606454
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
#
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203606886
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@ -
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203607189
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@ -
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203607354
##
File path: runtime/driver/src/main/java/edu/snu/nemo/driver/NemoDriver.java
#
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203607677
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@ -
wynot12 commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203611605
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
wynot12 commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203611915
##
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.ja
wonook commented on a change in pull request #73: [NEMO-62] Support Multiple
DAG Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203615620
##
File path: client/src/main/java/edu/snu/nemo/client/JobLauncher.java
##
@@ -1
wonook commented on issue #73: [NEMO-62] Support Multiple DAG Submission in a
Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#issuecomment-406170856
@seojangho @wynot12 Thanks! Can you check and comment if there are further
changes that has to be made?
--
wonook closed pull request #69: [NEMO-2, 106, 117] Remove the /tests directory
URL: https://github.com/apache/incubator-nemo/pull/69
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 for
wonook edited a comment on issue #73: [NEMO-62] Support Multiple DAG Submission
in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#issuecomment-406170856
@seojangho @wynot12 Thanks! Can you check and comment if there are further
changes that has to be made?
F
wonook edited a comment on issue #73: [NEMO-62] Support Multiple DAG Submission
in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#issuecomment-406170856
@seojangho @wynot12 Thanks! Can you check and comment if there are further
changes that has to be made?
F
seojangho opened a new pull request #79: [NEMO-164] Reduce Overhead of
FreeSlotSchedulingConstraint
URL: https://github.com/apache/incubator-nemo/pull/79
JIRA: [NEMO-164: Reduce Overhead of
FreeSlotSchedulingConstraint](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-164)
*
sanha closed pull request #79: [NEMO-164] Reduce Overhead of
FreeSlotSchedulingConstraint
URL: https://github.com/apache/incubator-nemo/pull/79
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 t
johnyangk opened a new pull request #80: [NEMO-165] Bug when a task reads from
multiple parent tasks
URL: https://github.com/apache/incubator-nemo/pull/80
JIRA: [NEMO-165: Bug when a task reads from multiple parent
tasks](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-165)
jeongyooneo closed pull request #80: [NEMO-165] Bug when a task reads from
multiple parent tasks
URL: https://github.com/apache/incubator-nemo/pull/80
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 closed pull request #73: [NEMO-62] Support Multiple DAG Submission in
a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73
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
shpark closed pull request #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76
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
shpark commented on issue #76: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/76#issuecomment-406452879
Close. Another pull request regarding the same issue will be ready soon.
--
shpark opened a new pull request #81: [NEMO-151] Add OutputWriters for
additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81
JIRA: [NEMO-151: Add OutputWriters for addtional tagged
outputs](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-151)
**Maj
seojangho commented on a change in pull request #81: [NEMO-151] Add
OutputWriters for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#discussion_r203940293
##
File path:
compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletim
johnyangk commented on a change in pull request #81: [NEMO-151] Add
OutputWriters for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#discussion_r203941717
##
File path:
compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletim
shpark commented on a change in pull request #81: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#discussion_r203945218
##
File path:
compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletime/r
shpark commented on a change in pull request #81: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#discussion_r203946134
##
File path:
compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletime/r
shpark commented on a change in pull request #81: [NEMO-151] Add OutputWriters
for additional tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#discussion_r203946134
##
File path:
compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/pass/compiletime/r
shpark commented on issue #81: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#issuecomment-406499571
Thanks for the reviews. I will reflect your corrections and suggestions into
this work. Thanks!
--
seojangho opened a new pull request #82: [NEMO-166] Do not attach
NodeNamesProperty to Source Stages
URL: https://github.com/apache/incubator-nemo/pull/82
JIRA: [NEMO-166: Do not attach NodeNamesProperty to Source
Stages](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-166)
sanha closed pull request #82: [NEMO-166] Do not attach NodeNamesProperty to
Source Stages
URL: https://github.com/apache/incubator-nemo/pull/82
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
shpark commented on issue #81: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#issuecomment-406963530
Now additional output execution property goes to edges instead of vertices,
thus we don't need to update execution property
johnyangk commented on issue #81: [NEMO-151] Add OutputWriters for additional
tagged outputs
URL: https://github.com/apache/incubator-nemo/pull/81#issuecomment-407239167
Looks good to me. I've approved the PR.
@seojangho can you take a look, and approve/merge?
-
johnyangk opened a new pull request #83: [NEMO-167] Rename Passes and Execution
Properties
URL: https://github.com/apache/incubator-nemo/pull/83
JIRA: [NEMO-167: Rename Passes and Execution
Properties](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-167)
**Major changes:**
johnyangk opened a new pull request #84: [NEMO-171] Fix confusing ITCase
input/output file names
URL: https://github.com/apache/incubator-nemo/pull/84
JIRA: [NEMO-171: Fix confusing ITCase input/output file
names](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-171)
**Major
jeongyooneo commented on a change in pull request #84: [NEMO-171] Fix confusing
ITCase input/output file names
URL: https://github.com/apache/incubator-nemo/pull/84#discussion_r207773921
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
##
jeongyooneo commented on a change in pull request #84: [NEMO-171] Fix confusing
ITCase input/output file names
URL: https://github.com/apache/incubator-nemo/pull/84#discussion_r207774259
##
File path: examples/spark/src/test/java/edu/snu/nemo/examples/spark/MRJava.java
##
701 - 800 of 1748 matches
Mail list logo