Github user sohum2002 commented on the issue:
https://github.com/apache/spark/pull/19454
Thank you all for your comments. I hope to improve in my future PRs. Cheers!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19454
Honestly I don't think it is worth doing this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19454
I actually think this can be confusing on Dataset[T], when the Dataset is
just untyped and a DataFrame. Do we throw a runtime exception there?
---
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19454
Is this worth doing?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user sohum2002 commented on the issue:
https://github.com/apache/spark/pull/19454
@HyukjinKwon - Thank you for your comments and analysis of this PR. I will
also try to improve the `flatMap(identity)` as mentioned by @srowen. Also, will
add a python implementation.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19454
BTW, for the answer to
https://github.com/apache/spark/pull/19454#issuecomment-335138642, I think you
should take a look at, for example, `flatMap` as a reference in `rdd.py` and
related tests,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19454
I think @srowen requested to fix it in a more performant way as well, for
example, referring https://github.com/apache/spark/pull/16276, if I understood
correctly and otherwise closing it.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19454
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/19454
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82550/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19454
**[Test build #82550 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82550/testReport)**
for PR 19454 at commit
[`cc08623`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19454
**[Test build #82550 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82550/testReport)**
for PR 19454 at commit
[`cc08623`](https://github.com/apache/spark/commit/cc
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19454
Let's fix up the PR title from `[SPARK-18855 ][SQL]` to
`[SPARK-18855][SQL]` BTW.
---
-
To unsubscribe, e-mail: reviews-unsu
Github user sohum2002 commented on the issue:
https://github.com/apache/spark/pull/19454
Would appreciate some help in the Python implementation of the `flatten`
function as I have never used pyspark. Could someone help me out?
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19454
Merged build finished. Test FAILed.
---
-
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/19454
**[Test build #82542 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82542/testReport)**
for PR 19454 at commit
[`261e45a`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19454
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82542/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19454
**[Test build #82542 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82542/testReport)**
for PR 19454 at commit
[`261e45a`](https://github.com/apache/spark/commit/26
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19454
This is missing from Python and Java. It also doesn't bother to implement
this more efficiently than flatMap(identity). I am not sure this is worth while?
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19454
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82541/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19454
**[Test build #82541 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82541/testReport)**
for PR 19454 at commit
[`075e7ef`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19454
Merged build finished. Test FAILed.
---
-
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/19454
**[Test build #82541 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82541/testReport)**
for PR 19454 at commit
[`075e7ef`](https://github.com/apache/spark/commit/07
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19454
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19454
Could you please add test cases?
---
-
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/19454
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
25 matches
Mail list logo