[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9687 --- 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] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-16 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-165253314 Thanks, merging 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

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164997732 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164997667 **[Test build #47785 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47785/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164997730 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

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164963096 In the commits above, I removed the `stripSparkFilter` function to share this. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164965009 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164965003 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

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164996396 cc @marmbrus --- 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] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164957946 The function I mentioned is moved to `SQLTestUtils` in another PR. So I will add a commit for this soon. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164975689 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

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-164979121 **[Test build #47785 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47785/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163081502 I will add that function to `SharedSQLContext` if it is okay. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163106415 **[Test build #47395 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47395/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163106474 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163089723 In this commit, I renamed the function extractSourceRDDToDataFrame to stripSparkFilter. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163089919 **[Test build #47395 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47395/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-163104687 Can I add the function `stripSparkFilter` to `SQLTestUtils` for `ParquetFilterSuite` and `OrcQuerySuite` (and possibly `OrcFilterSuite` I will make)? --- If your

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-17 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-157561940 Several things to mention. Firstly, I wonder if it is okay to put `extractSourceRDDToDataFrame` at `QueryTest`. I did not put but I think the

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-17 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-157564511 cc @liancheng --- 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] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156883379 **[Test build #45965 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45965/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156895666 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

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156895671 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156895218 **[Test build #45965 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45965/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156357679 **[Test build #45845 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45845/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156389724 **[Test build #45845 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45845/consoleFull)** for PR 9687 at commit

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156390050 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156390044 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

[GitHub] spark pull request: [SPARK-11677][SQL ]ORC filter tests all pass i...

2015-11-12 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/9687 [SPARK-11677][SQL ]ORC filter tests all pass if filters are actually not pushed down. Currently ORC filters are not tested properly. All the tests pass even if the filters are not pushed down

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156356500 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-11677][SQL] ORC filter tests all pass i...

2015-11-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9687#issuecomment-156356492 Merged build triggered. --- 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