Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/11714
---
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 feature is
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200375745
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200375747
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200375515
**[Test build #53936 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53936/consoleFull)**
for PR 11714 at commit
[`1a59fcf`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200344288
**[Test build #53936 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53936/consoleFull)**
for PR 11714 at commit
[`1a59fcf`](https://gi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57153635
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +48,24 @@ abstract class RuleExecutor[TreeType
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57153827
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -26,6 +26,9 @@ import org.apache.spark.sql.catalyst.util.
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57153533
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -26,6 +26,9 @@ import org.apache.spark.sql.catalyst.util._
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57153024
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -26,6 +26,9 @@ import org.apache.spark.sql.catalyst.util.
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57118618
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +48,24 @@ abstract class RuleExecutor[TreeType <:
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57118567
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +48,24 @@ abstract class RuleExecutor[TreeType <:
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r57118551
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -26,6 +26,9 @@ import org.apache.spark.sql.catalyst.util._
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200126124
I see. Sure. Let me know if anything I can help. Thanks!
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200086023
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200086020
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200085608
**[Test build #53834 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53834/consoleFull)**
for PR 11714 at commit
[`23a6cfc`](https://g
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200085464
I guess I would prioritize getting this in so that we make sure we don't
continue to introduce conflicting rules. If including #11828 in this PR is the
fastest path t
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200069678
@rxin @marmbrus This test build will fail for sure.
The related PRs is https://github.com/apache/spark/pull/11828 by @davies
Now, I think his version is mor
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200066870
**[Test build #53834 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53834/consoleFull)**
for PR 11714 at commit
[`23a6cfc`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200063012
Sure, will do it 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 t
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-200034759
Can you bring this up to date? We can disable rules that make this fail.
@marmbrus and I talked a bit and thought this is pretty critical to have.
---
If your proje
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196838065
@rxin Yeah. There is another one issue. In the predicate push down, we
should not push down any predicate if the child's Constraints already contain
them. Will submi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196722804
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196722803
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196722691
**[Test build #53176 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53176/consoleFull)**
for PR 11714 at commit
[`23663fe`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196720214
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196720217
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196719935
**[Test build #53175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53175/consoleFull)**
for PR 11714 at commit
[`1281f36`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196702896
**[Test build #53176 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53176/consoleFull)**
for PR 11714 at commit
[`23663fe`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196701472
I just merged #11682. Are there more?
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196699484
**[Test build #53175 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53175/consoleFull)**
for PR 11714 at commit
[`1281f36`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196698564
I will not trigger more tests in this PR until the related PR is merged.
Thanks!
---
If your project is set up for it, you can reply to this email and have your
rep
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196698066
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196698060
**[Test build #53173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53173/consoleFull)**
for PR 11714 at commit
[`f351362`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196698065
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196697886
**[Test build #53173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53173/consoleFull)**
for PR 11714 at commit
[`f351362`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196693818
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196693822
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196693517
**[Test build #53166 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53166/consoleFull)**
for PR 11714 at commit
[`797aabb`](https://g
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r5611
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -29,6 +29,7 @@ import org.apache.sp
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56122167
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56122162
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +47,23 @@ abstract class RuleExecutor[TreeType
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119880
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <: Tre
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119782
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <:
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119755
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <:
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119467
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <: Tre
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119479
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -29,6 +29,7 @@ import org.apache.spark.sq
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119436
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -46,15 +47,23 @@ abstract class RuleExecutor[TreeType <: Tr
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11714#discussion_r56119362
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/RuleExecutor.scala
---
@@ -98,7 +107,8 @@ abstract class RuleExecutor[TreeType <: Tre
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11714#issuecomment-196676820
**[Test build #53166 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53166/consoleFull)**
for PR 11714 at commit
[`797aabb`](https://gi
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/11714
[SPARK-13891] [SQL] [TEST] Issue Exceptions when Hitting the Max Iteration
Limit in Optimizer and Analyzer
What changes were proposed in this pull request?
This PR is to issue an e
51 matches
Mail list logo