[GitHub] spark pull request: [SPARK-12409][SQL]JDBC AND operator push down

2015-12-18 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10369#issuecomment-165716302 Can you combine and, or, in, and maybe other missing filters into a single PR? Every time I merge one the others conflict. --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-12409][SQL]JDBC AND operator push down

2015-12-18 Thread huaxingao
Github user huaxingao closed the pull request at: https://github.com/apache/spark/pull/10369 --- 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-12409][SQL]JDBC AND operator push down

2015-12-18 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/10369#issuecomment-165850180 Close for now. Will put the filter changes in one PR so it's easier to merge. Sorry for the inconvenience. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-12409][SQL]JDBC AND operator push down

2015-12-18 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/10369#issuecomment-165848876 Sorry for the trouble. I should have everything in one PR. Will do it now. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-12409][SQL]JDBC AND operator push down

2015-12-17 Thread huaxingao
GitHub user huaxingao opened a pull request: https://github.com/apache/spark/pull/10369 [SPARK-12409][SQL]JDBC AND operator push down For simple AND such as select * from test where THEID = 1 AND NAME = 'fred', The filters pushed down to JDBC layers are EqualTo(THEID,1),

[GitHub] spark pull request: [SPARK-12409][SQL]JDBC AND operator push down

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