[GitHub] spark issue #20413: [SPARK-23245][SS][TESTS] Don't access `lastExecution.exe...

2018-01-26 Thread zsxwing
Github user zsxwing commented on the issue: https://github.com/apache/spark/pull/20413 Thanks! Merging to master and 2.3. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #20413: [SPARK-23245][SS][TESTS] Don't access `lastExecution.exe...

2018-01-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20413 **[Test build #4079 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4079/testReport)** for PR 20413 at commit

[GitHub] spark issue #20413: [SPARK-23245][SS][TESTS] Don't access `lastExecution.exe...

2018-01-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20413 **[Test build #4079 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4079/testReport)** for PR 20413 at commit

[GitHub] spark issue #20413: [SPARK-23245][SS][TESTS] Don't access `lastExecution.exe...

2018-01-26 Thread jose-torres
Github user jose-torres commented on the issue: https://github.com/apache/spark/pull/20413 +1. I originally wrote this line, and I'm reasonably confident that (as indicated by the comment) I didn't intend to check anything other than the nullity of lastExecution. I continue to be

[GitHub] spark issue #20413: [SPARK-23245][SS][TESTS] Don't access `lastExecution.exe...

2018-01-26 Thread tdas
Github user tdas commented on the issue: https://github.com/apache/spark/pull/20413 LGTM. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: