[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-25 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10473#issuecomment-167214192 @xguo27 see my comment on the JIRA: https://issues.apache.org/jira/browse/SPARK-12521 I think this is intended, and should not be altered. --- If your

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-25 Thread xguo27
Github user xguo27 closed the pull request at: https://github.com/apache/spark/pull/10473 --- 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-12521][SQL][WIP] JDBCRelation does not ...

2015-12-25 Thread xguo27
Github user xguo27 commented on the pull request: https://github.com/apache/spark/pull/10473#issuecomment-167258976 Thanks @hvanhovell for clarifying it up. I will close this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-24 Thread xguo27
GitHub user xguo27 opened a pull request: https://github.com/apache/spark/pull/10473 [SPARK-12521][SQL][WIP] JDBCRelation does not honor lowerBound/upperBound JDBCRelation is not bounding the rows when lowerBound/upperBound are given. This change honors the bounds given. You can

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

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

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-24 Thread xguo27
Github user xguo27 commented on the pull request: https://github.com/apache/spark/pull/10473#issuecomment-167190872 Marking it [WIP] to invite discussion here. : ) As I suspect the original code includes infinity on both smaller than side and greater than side for a reason. --- If