Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13380
Merging with master, branch-2.0
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13380
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60994/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13380
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/13380
**[Test build #60994 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60994/consoleFull)**
for PR 13380 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13380
LGTM pending tests!
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #60994 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60994/consoleFull)**
for PR 13380 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13380
Other than that, this looks ready to me.
---
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/13380
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60721/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13380
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/13380
**[Test build #60721 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60721/consoleFull)**
for PR 13380 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13380
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/13380
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60720/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #60720 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60720/consoleFull)**
for PR 13380 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #60721 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60721/consoleFull)**
for PR 13380 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #60720 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60720/consoleFull)**
for PR 13380 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13380
@jkbradley Let me change it now. 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
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13380
this looks good to me other than deprecating context() and calling the new
method session()
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13380
+1 for deprecating ```context()``` now and calling the new method
```session()```
Sorry for the slow response!
---
If your project is set up for it, you can reply to this email and have
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/13380
FWIW, I think since `SQLContext` is deprecated, we should deprecate the
`context` methods in `ReadWrite`, and create new methods called `session` (or
`withSession`?) for clarity (since `context` is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13380
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/13380
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60468/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #60468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60468/consoleFull)**
for PR 13380 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #60468 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60468/consoleFull)**
for PR 13380 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13380
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 wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13380
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/13380
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59800/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #59800 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59800/consoleFull)**
for PR 13380 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13380
**[Test build #59800 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59800/consoleFull)**
for PR 13380 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13380
Those are the only items I'd modify. Basically, I like switching to
SparkSession but also maintaining compatibility for public APIs. Changing the
private APIs is fine.
---
If your project is
29 matches
Mail list logo