Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
@sun-rui Let me know if you are unable to do so. We need this in 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
Github user sun-rui commented on the issue:
https://github.com/apache/spark/pull/14175
ok, will submit another PR for 2.0 branch
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14175
Merged to master, but it doesn't pick cleanly into 2.0, and the conflict in
the tests wasn't entirely trivial. You can open another PR if it's important.
---
If your project is set up for it, you
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
Thanks @sun-rui !
---
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 wishes so,
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
@srowen LGTM. Can you merge this into 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63345/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
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/14175
**[Test build #63345 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63345/consoleFull)**
for PR 14175 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #63345 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63345/consoleFull)**
for PR 14175 at commit
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
A couple minor style issues then LGTM. Will ping the committer when style
issues are resolved.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63327/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
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/14175
**[Test build #63327 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63327/consoleFull)**
for PR 14175 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #63327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63327/consoleFull)**
for PR 14175 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
Merged build finished. Test FAILed.
---
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/14175
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63321/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #63321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63321/consoleFull)**
for PR 14175 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #63321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63321/consoleFull)**
for PR 14175 at commit
Github user sun-rui commented on the issue:
https://github.com/apache/spark/pull/14175
rebased to master
---
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 wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63312/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #63312 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63312/consoleFull)**
for PR 14175 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #63312 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63312/consoleFull)**
for PR 14175 at commit
Github user sun-rui commented on the issue:
https://github.com/apache/spark/pull/14175
@mgummelt, regression test case added. Not sure it is the expected one.
---
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 mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
thanks!
---
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 wishes so, or if the
Github user sun-rui commented on the issue:
https://github.com/apache/spark/pull/14175
@mgummelt, will do it soon
---
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 mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
@sun-rui are you going to get to this? Otherwise I'll take it over.
---
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 sun-rui commented on the issue:
https://github.com/apache/spark/pull/14175
Sure, will add it
---
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 wishes so,
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
Can you add a regression test?
Then LGTM
---
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
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14175
@sun-rui ping
---
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 wishes so, or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
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/14175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62514/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #62514 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62514/consoleFull)**
for PR 14175 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #62514 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62514/consoleFull)**
for PR 14175 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14175
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/14175
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62225/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #62225 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62225/consoleFull)**
for PR 14175 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14175
**[Test build #62225 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62225/consoleFull)**
for PR 14175 at commit
38 matches
Mail list logo
