[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218310415 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-12639] [SQL] Mark Filters Fully Handled...

2016-05-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218310416 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-12639] [SQL] Mark Filters Fully Handled...

2016-05-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218310193 **[Test build #58268 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58268/consoleFull)** for PR 11317 at commit [`117999f`](https://g

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-10 Thread jacek-lewandowski
Github user jacek-lewandowski commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218287519 jenkins, test 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

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218287739 **[Test build #58268 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58268/consoleFull)** for PR 11317 at commit [`117999f`](https://gi

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-10 Thread jacek-lewandowski
Github user jacek-lewandowski commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218287442 test 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-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread RussellSpitzer
Github user RussellSpitzer commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218052162 I don't think this is because of me ``` # A fatal error has been detected by the Java Runtime Environment: # # Internal Error (sharedRuntime.

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218040523 Test FAILed. 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-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218040522 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] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218040442 **[Test build #58192 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58192/consoleFull)** for PR 11317 at commit [`117999f`](https://g

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread RussellSpitzer
Github user RussellSpitzer commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218030539 @HyukjinKwon + @yhuai Sorry it took so long! Things have been busy :) --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218031170 @RussellSpitzer Thanks for bearing with my pings here and there! --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-218030822 **[Test build #58192 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58192/consoleFull)** for PR 11317 at commit [`117999f`](https://gi

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-05-06 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-217602140 @RussellSpitzer I saw you answered my ping before. Excuse my ping here again. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-03-19 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-197597601 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 enab

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-197605186 **[Test build #53374 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53374/consoleFull)** for PR 11317 at commit [`c33daae`](https://g

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-197605244 Test FAILed. 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-12639] [SQL] Mark Filters Fully Handled...

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-197605241 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] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-03-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-197599125 **[Test build #53374 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53374/consoleFull)** for PR 11317 at commit [`c33daae`](https://gi

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-03-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-197171280 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 p

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187842535 Test FAILed. 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-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187842533 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] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187842476 **[Test build #51781 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51781/consoleFull)** for PR 11317 at commit [`c33daae`](https://g

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187836927 **[Test build #51781 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51781/consoleFull)** for PR 11317 at commit [`c33daae`](https://gi

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187835970 Thank you @RussellSpitzer! It looks good. --- 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

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/11317#discussion_r53827152 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala --- @@ -562,14 +566,15 @@ private[sql] object DataSource

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187833262 @RussellSpitzer If you get a chance, can you add an example showing the plan before and after this change? --- If your project is set up for it, you can reply to this em

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-23 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187832397 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 enab

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187541245 cc @yhuai --- 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 enable

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11317#issuecomment-187490084 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 p

[GitHub] spark pull request: [SPARK-12639] [SQL] Mark Filters Fully Handled...

2016-02-22 Thread RussellSpitzer
GitHub user RussellSpitzer opened a pull request: https://github.com/apache/spark/pull/11317 [SPARK-12639] [SQL] Mark Filters Fully Handled By Sources with * ## What changes were proposed in this pull request? In order to make it clear which filters are fully handled by the

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-02-22 Thread RussellSpitzer
Github user RussellSpitzer closed the pull request at: https://github.com/apache/spark/pull/10929 --- 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 featur

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175874599 @RussellSpitzer I just realized that this is against branch 1.6. Can you create a PR for master? --- If your project is set up for it, you can reply to this email and ha

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175789209 **[Test build #50206 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50206/consoleFull)** for PR 10929 at commit [`40a75a4`](https://g

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175789222 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] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175789226 Test FAILed. 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-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175787113 **[Test build #50206 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50206/consoleFull)** for PR 10929 at commit [`40a75a4`](https://gi

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175781480 test 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 featur

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175780484 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] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175780474 **[Test build #50197 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50197/consoleFull)** for PR 10929 at commit [`40a75a4`](https://g

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175780486 Test FAILed. 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-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175772895 **[Test build #50197 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50197/consoleFull)** for PR 10929 at commit [`40a75a4`](https://gi

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175767322 @RussellSpitzer Thank you for the PR! This looks good! It will be great if you have time to add before/after comparison in the description and change the title to `[SPARK

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-27 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175764743 test 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 featur

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10929#issuecomment-175275870 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 p

[GitHub] spark pull request: SPARK-12639 SQL Mark Filters Fully Handled By ...

2016-01-26 Thread RussellSpitzer
GitHub user RussellSpitzer opened a pull request: https://github.com/apache/spark/pull/10929 SPARK-12639 SQL Mark Filters Fully Handled By Sources with * In order to make it clear which filters are fully handled by the underlying datasource we will mark them with a *. This will g