[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-30 Thread rdblue
Github user rdblue commented on the pull request: https://github.com/apache/spark/pull/13389#issuecomment-222521471 +1 overall, good catch on those 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 do

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-30 Thread rdblue
Github user rdblue commented on a diff in the pull request: https://github.com/apache/spark/pull/13389#discussion_r65089323 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilters.scala --- @@ -50,7 +50,6 @@ private[sql] object ParquetFi

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-29 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13389#issuecomment-222346754 Hi @rdblue @liancheng , Could you please take a look? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13389#issuecomment-222346086 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 projec

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13389#issuecomment-222346088 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13389#issuecomment-222346061 **[Test build #59580 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59580/consoleFull)** for PR 13389 at commit [`04f24e3`](https://g

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13389#issuecomment-222343981 **[Test build #59580 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59580/consoleFull)** for PR 13389 at commit [`04f24e3`](https://gi

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-28 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/13389 [SPARK-9876][SQL][FOLLOWUP] Enable string and binary tests for Parquet predicate pushdown ## What changes were proposed in this pull request? It seems Parquet has been upgraded to 1.8.