Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22939
Error looks reasonable...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
@felixcheung, I tested when the user passes in a column that is not a
literal string, and it shows the results as below:
```
> json <- '{"name":"Bob"}'
> df <- sql("SELECT * FROM
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Thank you, @felixcheung for approving this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Hey @felixcheung, have you found some time to take a look for this please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Sure!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22939
Sorry for the delay, will do another pass in 1 or 2 days
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
gentle ping, @felixcheung.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Hey @felixcheung, it should be ready for another look.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98550 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98550/testReport)**
for PR 22939 at commit
[`7c8e226`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98550/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98550 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98550/testReport)**
for PR 22939 at commit
[`7c8e226`](https://github.com/apache/spark/commit/7c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4814/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98470/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98470 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98470/testReport)**
for PR 22939 at commit
[`c582757`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4767/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98470 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98470/testReport)**
for PR 22939 at commit
[`c582757`](https://github.com/apache/spark/commit/c5
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Let me clean up and deal with other comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98457/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98457 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98457/testReport)**
for PR 22939 at commit
[`c0a9384`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Will make another PR after this gets merged to allow the cases below:
```r
df <- sql("SELECT named_struct('name', 'Bob') as people")
df <- mutate(df, people_json = to_json(df
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98457 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98457/testReport)**
for PR 22939 at commit
[`c0a9384`](https://github.com/apache/spark/commit/c0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4757/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98449/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98449 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98449/testReport)**
for PR 22939 at commit
[`5f0a3b6`](https://github.com/apache/spark/commit/5
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
Makes sense. Let me separate it tomorrow.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22939
> In addition, it also proposes to make from_csv and from_json accept
structType, DDL-formatted string, DDL-formatted string literal, and
schema_of_[csv|json] as schema so that we can utilise both sc
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98449 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98449/testReport)**
for PR 22939 at commit
[`5f0a3b6`](https://github.com/apache/spark/commit/5f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4754/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98446/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98446 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98446/testReport)**
for PR 22939 at commit
[`5f0a3b6`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98442/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98441/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4752/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98446 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98446/testReport)**
for PR 22939 at commit
[`5f0a3b6`](https://github.com/apache/spark/commit/5f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98442/testReport)**
for PR 22939 at commit
[`52bae78`](https://github.com/apache/spark/commit/52
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4749/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22939
cc @felixcheung and @MaxGekk
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4748/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22939
**[Test build #98441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98441/testReport)**
for PR 22939 at commit
[`c4a78fc`](https://github.com/apache/spark/commit/c4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22939
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
55 matches
Mail list logo