Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16312
Merged into master, thanks for reviewing.
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/16312
LGTM. 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,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
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/16312
**[Test build #71013 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71013/testReport)**
for PR 16312 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71013/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #71013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71013/testReport)**
for PR 16312 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16312
@felixcheung @shivaram I changed ```read.ml``` to pass in
```sparkSession``` if necessary, then it will not use stale spark context which
will cause errors. This is consistent with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70984 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70984/testReport)**
for PR 16312 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
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/16312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70984/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70984/testReport)**
for PR 16312 at commit
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/16312
@shivaram @felixcheung You comments are really very helpful, I understand
the problem and will try to figure out some ways to fix it. Thanks very much.
---
If your project is set up for it, you
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16312
ah, thank you @shivaram. sorry I couldn't get around to investigate earlier.
@yanboliang It looks like that is the design in the trait BaseReadWrite
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16312
I looked at this more closely and I think I found the problem - Not sure
its easy to fix though.
What I traced here is:
- When we call sparkR.session.stop and sparkR.session the same JVM
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16312
sure, I'll take a deeper look in a day or 2.
---
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/16312
I'm trying to figure out the cause of this issue but still no progress
until now, could you or @shivaram give some hints or suggestion? Thanks.
---
If your project is set up for it, you can
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16312
Hmm, from what I can see pretty sure it used to work with
sparkR.session.stop() calls for enableHiveSupport = F sessions. Do we know why
this is suddenly causing issues?
---
If your project
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70310/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70310 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70310/testReport)**
for PR 16312 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
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/16312
**[Test build #70310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70310/testReport)**
for PR 16312 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
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/16312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70302/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70302 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70302/testReport)**
for PR 16312 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70302 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70302/testReport)**
for PR 16312 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16312
I like how they are grouped. not sure why tests are failing though
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70261/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70261 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70261/testReport)**
for PR 16312 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70261 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70261/testReport)**
for PR 16312 at commit
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/16312
retest this please.
---
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 wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/16312
This is quite necessary change! The file becomes very lengthy. 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16312
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/16312
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70256/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70256 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70256/testReport)**
for PR 16312 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16312
**[Test build #70256 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70256/testReport)**
for PR 16312 at commit
36 matches
Mail list logo