Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13962
LGTM. Merging to master / 2.0 (fix is simple enough and I'd rather have the
whole 2.x line behave the same here).
---
If your project is set up for it, you can reply to this email and have your
Github user renozhang commented on the issue:
https://github.com/apache/spark/pull/13962
@vanzin I've updated codes and comments.
Please take a look, thanks a lot.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13962
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61619/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13962
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13962
**[Test build #61619 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61619/consoleFull)**
for PR 13962 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13962
**[Test build #61619 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61619/consoleFull)**
for PR 13962 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/13962
Looks good to me, CC @vanzin about `Launcher` thing.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13962
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61474/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13962
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13962
**[Test build #61474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61474/consoleFull)**
for PR 13962 at commit
10 matches
Mail list logo