[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-28 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20073 Merged to master. Thank you @felixcheung. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For add

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-26 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/20073 Fair enough. Calling it, is good enough. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For ad

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20073 I tried to check if there is a easier way but not sure. Scala side seems checking the plan as well but I think I can't simply do it in R side and don't think it's worth .. --- ---

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-25 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/20073 Yes LGTM Btw is there a better way to test localCheckpoint that it is not writing to checkpointDir? --- - To

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20073 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85364/ Test PASSed. ---

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20073 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-24 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20073 **[Test build #85364 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85364/testReport)** for PR 20073 at commit [`ec27a80`](https://github.com/apache/spark/commit/e

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-24 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20073 **[Test build #85364 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85364/testReport)** for PR 20073 at commit [`ec27a80`](https://github.com/apache/spark/commit/ec

[GitHub] spark issue #20073: [SPARK-22843][R] Adds localCheckpoint in R

2017-12-24 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20073 cc @felixcheung, could you check if I understood your intention correctly? --- - To unsubscribe, e-mail: reviews-unsubscr...@