Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
Thanks @rxin @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18956
Thanks! Merged to master.
---
-
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/18956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81531/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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/18956
**[Test build #81531 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81531/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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/18956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81529/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81529 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81529/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81531 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81531/testReport)**
for PR 18956 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18956
LGTM except two minor comments
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81529 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81529/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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/18956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81518/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81518/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81518/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81504/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81504 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81504/testReport)**
for PR 18956 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81504 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81504/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81496/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81496 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81496/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81496 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81496/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81460/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81460/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81460 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81460/testReport)**
for PR 18956 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
#19050 is merged now. Let's see if there is any rule can fail this
structural integrity check.
---
-
To unsubscribe, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
`RewritePredicateSubquery` fails structural integrity check because it can
produce `Join` with conflicting attributes in its left and right plans.
I submitted #19050 to fix it.
---
If your
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
Seems there are other issues caused by `RewritePredicateSubquery` rule.
I'll investigate and fix it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81079/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81079 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81079/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #81079 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81079/testReport)**
for PR 18956 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
#18968 is merged. This should pass the tests now.
---
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
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
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 wishes so,
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
The PR going to fix the issue in `PullupCorrelatedPredicates` is submitted
at #18968.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
The reason `PullupCorrelatedPredicates` produces unresolved plan:
The query causing the problem in `subquery_in_having.q` looks like:
select b.key, min(b.value)
from src
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18956
Interesting, existing `PullupCorrelatedPredicates` produces unresolved
plan. I'll figure out the reason.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80718/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80718 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80718/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80717 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80717/testReport)**
for PR 18956 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80717/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80715/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18956
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80715 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80715/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80718/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80717 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80717/testReport)**
for PR 18956 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18956
**[Test build #80715 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80715/testReport)**
for PR 18956 at commit
53 matches
Mail list logo
