Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17711
Thanks! Merging to master.
---
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 w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76856/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76856 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76856/testReport)**
for PR 17711 at commit
[`de89791`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76856 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76856/testReport)**
for PR 17711 at commit
[`de89791`](https://github.com/apache/spark/commit/de
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
Jenkins, retest this please.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76850/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76850/testReport)**
for PR 17711 at commit
[`de89791`](https://github.com/apache/spark/commit/de
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
@rxin ok, thank for the suggestion!
---
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
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17711
I feel both are pretty complicated. Can we just do something similar to
CombineUnion:
```
/**
* Combines all adjacent [[Union]] operators into a single [[Union]].
*/
object Co
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17711
@maropu The solution using `tailrec` looks more straightforward. Could you
submit the PR based on that? Thanks!
---
If your project is set up for it, you can reply to this email and have your
re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76840/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76840 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76840/testReport)**
for PR 17711 at commit
[`089db30`](https://github.com/apache/spark/commit/0
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
I quickly brushed up the Optimizer code based on your advice:
Using `Stack`:
https://github.com/apache/spark/commit/a17d933aaad5c38a9d9ff20f978c1bad6c774fb1#diff-a1acb054bc376603ef510e6d0e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76840 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76840/testReport)**
for PR 17711 at commit
[`089db30`](https://github.com/apache/spark/commit/08
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17711
LGTM pending minor comment.
We need to add an extra optimizer rule to combine the adjacent concatenate
expressions. Thanks!
---
If your project is set up for it, you can reply to this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76795/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76795 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76795/testReport)**
for PR 17711 at commit
[`96db575`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76795 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76795/testReport)**
for PR 17711 at commit
[`96db575`](https://github.com/apache/spark/commit/96
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17711
Please follow the other optimizer rules. We need to add a optimizer test
suite. For example, `SimplifyConditionalSuite`
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76769/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76769 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76769/testReport)**
for PR 17711 at commit
[`c88652c`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76769 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76769/testReport)**
for PR 17711 at commit
[`c88652c`](https://github.com/apache/spark/commit/c8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76723/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76723 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76723/testReport)**
for PR 17711 at commit
[`8890b94`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76723 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76723/testReport)**
for PR 17711 at commit
[`8890b94`](https://github.com/apache/spark/commit/88
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76657/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76657 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76657/testReport)**
for PR 17711 at commit
[`cb4b26e`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76657 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76657/testReport)**
for PR 17711 at commit
[`cb4b26e`](https://github.com/apache/spark/commit/cb
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
Jenkins, retest this please.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76626/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76626 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76626/testReport)**
for PR 17711 at commit
[`cb4b26e`](https://github.com/apache/spark/commit/cb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76606/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76606 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76606/testReport)**
for PR 17711 at commit
[`96293f3`](https://github.com/apache/spark/commit/9
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
The R test failure seemed to be fixed in
https://github.com/apache/spark/commit/2abfee18b6511482b916c36f00bf3abf68a59e19
---
If your project is set up for it, you can reply to this email and have yo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76606 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76606/testReport)**
for PR 17711 at commit
[`96293f3`](https://github.com/apache/spark/commit/96
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
This failure seems unrelated to this pr? (other prs also hit the same R
test failure...).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
Jenkins, retest this please.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76597/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76597/testReport)**
for PR 17711 at commit
[`96293f3`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76597 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76597/testReport)**
for PR 17711 at commit
[`96293f3`](https://github.com/apache/spark/commit/96
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
Jenkins, retest this please.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76580/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76580 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76580/testReport)**
for PR 17711 at commit
[`96293f3`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76580 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76580/testReport)**
for PR 17711 at commit
[`96293f3`](https://github.com/apache/spark/commit/96
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76578/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76578 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76578/testReport)**
for PR 17711 at commit
[`f984c6b`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76578 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76578/testReport)**
for PR 17711 at commit
[`f984c6b`](https://github.com/apache/spark/commit/f9
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
@hvanhovell ping
---
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 i
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
ping
---
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 featur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76030/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76030 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76030/testReport)**
for PR 17711 at commit
[`83242d5`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76030 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76030/testReport)**
for PR 17711 at commit
[`83242d5`](https://github.com/apache/spark/commit/83
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76020/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76020 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76020/testReport)**
for PR 17711 at commit
[`72d1ae1`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76020 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76020/testReport)**
for PR 17711 at commit
[`72d1ae1`](https://github.com/apache/spark/commit/72
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
Jenkins, retest this please.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76016/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76014/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76014 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76014/testReport)**
for PR 17711 at commit
[`0701f87`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76016 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76016/testReport)**
for PR 17711 at commit
[`72d1ae1`](https://github.com/apache/spark/commit/72
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76014/testReport)**
for PR 17711 at commit
[`0701f87`](https://github.com/apache/spark/commit/07
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76010/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76010 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76010/testReport)**
for PR 17711 at commit
[`9cfaef6`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76010/testReport)**
for PR 17711 at commit
[`9cfaef6`](https://github.com/apache/spark/commit/9c
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
okay, I'll add soon.
---
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,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76009/testReport)**
for PR 17711 at commit
[`bd36e58`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76009/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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/17711
**[Test build #76009 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76009/testReport)**
for PR 17711 at commit
[`bd36e58`](https://github.com/apache/spark/commit/bd
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17711
can you add a test case in sql query file tests?
---
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 feat
87 matches
Mail list logo