Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13634
Thank you, @vanzin !
---
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
enabled and
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13634
LGTM. Merging to master / 2.0.
---
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
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13634
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60397/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13634
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/13634
**[Test build #60397 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60397/consoleFull)**
for PR 13634 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13634
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13634
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60394/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13634
**[Test build #60394 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60394/consoleFull)**
for PR 13634 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13634
Thank you always, @srowen .
---
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13634
**[Test build #60397 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60397/consoleFull)**
for PR 13634 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13634
**[Test build #60394 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60394/consoleFull)**
for PR 13634 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13634
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60383/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13634
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/13634
**[Test build #60383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60383/consoleFull)**
for PR 13634 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13634
Thank you for review, @srowen .
Oh, right. That sounds much better to me.
I'll update this PR like that.
---
If your project is set up for it, you can reply to this email and have
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13634
Do you need the flag anymore? Maybe return an Option with the error instead
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13634
**[Test build #60383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60383/consoleFull)**
for PR 13634 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13634
Hi, @vanzin .
Could you review this when you have some time?
---
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
18 matches
Mail list logo
