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,
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 *
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
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:
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:
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:
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:
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,
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
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
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
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
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):
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
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
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
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):
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
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
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
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
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
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 =
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
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):
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
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
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
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
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
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
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
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):
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:
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
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
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
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
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
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):
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
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
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):
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
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,
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):
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
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
55 matches
Mail list logo
