Github user ptkool commented on the issue:
https://github.com/apache/spark/pull/23222
I'm not sure how this ended up being omitted. `TransposeWindowSuite` will
be fine since it creates a simple optimizer from this rule and a few others.
The new test added to `DataFrameWindowFunctionsS
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23222
Jenkins passes, which means the previously added end-to-end test can't not
show the benefit of this rule. We should update it.
---
---
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/23222
Shall we add a SQL tag to the title?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99701/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
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/23222
**[Test build #99701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99701/testReport)**
for PR 23222 at commit
[`1270e89`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
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/23222
**[Test build #99701 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99701/testReport)**
for PR 23222 at commit
[`1270e89`](https://github.com/apache/spark/commit/12
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
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/5748/
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23222
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99698/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
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/23222
**[Test build #99698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99698/testReport)**
for PR 23222 at commit
[`1270e89`](https://github.com/apache/spark/commit/1
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/23222
We can compare the plans and see whether the rule takes an effect.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23222
That PR also added an end-to-end test, does this mean that test is not
valid?
---
-
To unsubscribe, e-mail: reviews-unsubscr..
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
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/5747/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23222
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/23222
**[Test build #99698 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99698/testReport)**
for PR 23222 at commit
[`1270e89`](https://github.com/apache/spark/commit/12
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/23222
cc @ptkool @jiangxb1987 @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
19 matches
Mail list logo