[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810794 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -252,18

[GitHub] sanha closed pull request #112: [NEMO-179] Delayed Task Cloning

sanha closed pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112 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] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810367 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -204,23

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211809651 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/BlockManagerMaster.java ## @@ -302,21

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211809145 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/BlockManagerMaster.java ## @@ -85,6 +86,7

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211808474 ## File path: examples/beam/src/test/java/edu/snu/nemo/examples/beam/policy/SpeculativeExecutionPolicyParallelismFive.java

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810326 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -169,16

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211807844 ## File path:

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211808263 ## File path: compiler/optimizer/src/main/java/edu/snu/nemo/compiler/optimizer/policy/BasicPullPolicy.java ## @@

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810943 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java ## @@ -103,6 +106,15

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810810 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -252,18

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211807022 ## File path: common/src/main/java/edu/snu/nemo/common/ir/vertex/executionproperty/ClonedSchedulingProperty.java

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810869 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -422,6

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211807988 ## File path:

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211810707 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -252,18

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211817316 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -252,18

[GitHub] johnyangk commented on issue #112: [NEMO-179] Delayed Task Cloning

johnyangk commented on issue #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#issuecomment-414892660 Thanks @sanha for your review. I've updated the PR and added pointers to JIRA issues where appropriate.

[GitHub] johnyangk commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

johnyangk commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211816955 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/PlanStateManager.java ## @@ -252,18

[GitHub] johnyangk commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

johnyangk commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211816796 ## File path: runtime/master/src/main/java/edu/snu/nemo/runtime/master/BlockManagerMaster.java ## @@ -302,21

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211817451 ## File path:

[GitHub] sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning

sanha commented on a change in pull request #112: [NEMO-179] Delayed Task Cloning URL: https://github.com/apache/incubator-nemo/pull/112#discussion_r211817373 ## File path: