Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-92509822
Thanks! Merged 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5390
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-92006095
[Test build #30105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30105/consoleFull)
for PR 5390 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-92006098
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91997467
[Test build #30105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30105/consoleFull)
for PR 5390 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91928278
[Test build #30087 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30087/consoleFull)
for PR 5390 at commit
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91925973
Added test case. please test
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91927790
ok to test
---
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 pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91936716
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5390#discussion_r28199234
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetFilterSuite.scala
---
@@ -22,7 +22,8 @@ import parquet.filter2.predicate.Operators._
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91936711
[Test build #30087 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30087/consoleFull)
for PR 5390 at commit
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/5390#discussion_r28043391
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -214,6 +214,12 @@ private[sql] abstract class SparkStrategies
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/5390#discussion_r28043392
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -223,8 +229,12 @@ private[sql] abstract class SparkStrategies
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-91149021
Good catch! #5210 fixed this for the new Parquet data source but forgot the
old Parquet code path. Would you please to add a test case for this? Thanks!
---
If your
GitHub user saucam opened a pull request:
https://github.com/apache/spark/pull/5390
[SQL][SPARK-6742]: Don't push down predicates which reference partition
column(s)
cc @liancheng
You can merge this pull request into a Git repository by running:
$ git pull
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5390#issuecomment-90533449
Can one of the admins verify this patch?
---
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
16 matches
Mail list logo