Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14083
Thank you for starting and completing this! :D
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
Merging to master. Thanks for all the reviews!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
Done.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14083
@hvanhovell . Could you update the PR description benchmark result, too?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90252/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #90252 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90252/testReport)**
for PR 14083 at commit
[`cbc164d`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/2949/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
**[Test build #90252 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90252/testReport)**
for PR 14083 at commit
[`cbc164d`](https://github.com/apache/spark/commit/cb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/2947/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90250/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
**[Test build #90250 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90250/testReport)**
for PR 14083 at commit
[`ac4abcd`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #90250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90250/testReport)**
for PR 14083 at commit
[`ac4abcd`](https://github.com/apache/spark/commit/ac
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14083
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89637/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #89637 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89637/testReport)**
for PR 14083 at commit
[`eb6dd80`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/2533/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
**[Test build #89637 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89637/testReport)**
for PR 14083 at commit
[`eb6dd80`](https://github.com/apache/spark/commit/eb
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #89636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89636/testReport)**
for PR 14083 at commit
[`85e0744`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89636/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/2532/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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/14083
**[Test build #89636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89636/testReport)**
for PR 14083 at commit
[`85e0744`](https://github.com/apache/spark/commit/85
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
@cloud-fan can you take a look?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14083
Close it? @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14083
cc @hvanhovell
---
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, o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65231/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #65231 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65231/consoleFull)**
for PR 14083 at commit
[`a1e5312`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #65231 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65231/consoleFull)**
for PR 14083 at commit
[`a1e5312`](https://github.com/apache/spark/commit/a
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
@JoshRosen I have moved the implementation into `AttributeSeq`.
---
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 doe
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14083
Just came across the old PR. Because this is similar in spirit to #13505, I
wonder whether it might make sense to put some of this logic into
`AttributeSeq`:
https://github.com/apache/sp
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14083
Finally! Congrat!
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61952/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61952 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61952/consoleFull)**
for PR 14083 at commit
[`a5d1a4a`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61952 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61952/consoleFull)**
for PR 14083 at commit
[`a5d1a4a`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61923/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Merged build finished. Test FAILed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61923 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61923/consoleFull)**
for PR 14083 at commit
[`c75ae8d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61923/consoleFull)**
for PR 14083 at commit
[`c75ae8d`](https://github.com/apache/spark/commit/c
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14083
I've got another idea (orthogonal to this). Thank you, @hvanhovell .
---
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 pro
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14083
yea. :-)
---
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 fe
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
@viirya you mean I forgot to add `time(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 project does not have
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14083
The codes in the description seems incomplete?
---
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 featu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61886/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Merged build finished. Test FAILed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61886 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61886/consoleFull)**
for PR 14083 at commit
[`3e39720`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61886 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61886/consoleFull)**
for PR 14083 at commit
[`3e39720`](https://github.com/apache/spark/commit/3
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
messed up the merge - lemme fix that
---
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
ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61884/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Merged build finished. Test FAILed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61884 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61884/consoleFull)**
for PR 14083 at commit
[`a8a6a83`](https://github.com/apache/spark/commit/
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14083
Oh, it's strange. It still fails at compiling in my local laptop.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61884 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61884/consoleFull)**
for PR 14083 at commit
[`a8a6a83`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61882/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61882 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61882/consoleFull)**
for PR 14083 at commit
[`b4c8cdb`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14083
Merged build finished. Test FAILed.
---
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
e
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14083
Wow! And, thank you for pinging me. :)
---
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 featur
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14083
**[Test build #61882 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61882/consoleFull)**
for PR 14083 at commit
[`b4c8cdb`](https://github.com/apache/spark/commit/b
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14083
cc @dongjoon-hyun
---
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
67 matches
Mail list logo
