Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-222335566
Sorry, I did not notice this PR. I submitted another PR
(https://github.com/apache/spark/pull/13380) that removes `SQLContext` from
`MLlib`. Any reason why you still
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-16523
Oh, thank you, @andrewor14 . I see.
I will make another PR for using `builder.sparkContext(sc)` pattern.
---
If your project is set up for it, you can reply t
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13352
---
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 feature is ena
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-15279
@dongjoon-hyun actually the `builder.sparkContext` method was added
recently so there are other existing places where we could use that. Would you
mind submitting a
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13352#discussion_r64942596
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/api/python/PythonMLLibAPI.scala ---
@@ -1178,8 +1176,9 @@ private[python] class PythonMLLibAPI exten
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13352#discussion_r64942547
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/api/python/PythonMLLibAPI.scala ---
@@ -1178,8 +1176,9 @@ private[python] class PythonMLLibAPI exten
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-06324
Hi, @andrewor14 .
This is about deprecation warnings about `SQLContext`s.
---
If your project is set up for it, you can reply to this email and have your
repl
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-222115643
This removes lots of deprecation warning messages like the followings.
```
/home/jenkins/workspace/SparkPullRequestBuilder/mllib/src/main/scala/org/apache/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-222089522
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-222089523
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-222089414
**[Test build #59475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59475/consoleFull)**
for PR 13352 at commit
[`ae8acae`](https://g
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/13352
[SPARK-15603][MLLIB] Replace SQLContext with SparkSession in ML/MLLib
## What changes were proposed in this pull request?
This PR replaces all deprecated `SQLContext` occurrences with
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13352#issuecomment-222082414
**[Test build #59475 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59475/consoleFull)**
for PR 13352 at commit
[`ae8acae`](https://gi
13 matches
Mail list logo