Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14195
Adding more tests in new PR. I'm hoping to remove deprecated functions and
this routing mechanism as soon as 2.0.0 is released...
---
If your project is set up for it, you can reply to this
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14195
Great. This is 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 project does not have this feature
enabled and
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14195
I guess it would be good to have an additional test case, but I'm going to
merge this to make sure this makes 2.0 RCs. LGTM. We could look at ways to
make this more robust in the future
---
If
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14195
What I wanted is `SparkR::sql`.
---
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14195
Yep. I mean . :)
```
SparkR::sql("select 1")
```
---
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
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14195
Thanks. That actually would fail - can't run an empty sql query.
---
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14195
Hi, @felixcheung
LGTM. I tested it locally, too.
By the way, could you add the original reporter's testcase, too?
```
SparkR::sql("")
```
---
If your project is set up
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14195
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62296/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14195
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/14195
**[Test build #62296 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62296/consoleFull)**
for PR 14195 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14195
**[Test build #62296 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62296/consoleFull)**
for PR 14195 at commit
11 matches
Mail list logo
