[GitHub] spark pull request #18310: [SPARK-21103][SQL] QueryPlanConstraints should be...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/18310 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #18310: [SPARK-21103][SQL] QueryPlanConstraints should be...
GitHub user rxin reopened a pull request: https://github.com/apache/spark/pull/18310 [SPARK-21103][SQL] QueryPlanConstraints should be part of LogicalPlan ## What changes were proposed in this pull request? QueryPlanConstraints should be part of LogicalPlan, rather than QueryPlan, since the constraint framework is only used for query plan rewriting and not for physical planning. ## How was this patch tested? Should be covered by existing tests, since it is a simple refactoring. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark SPARK-21103 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/18310.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #18310 commit 90cff62eaffbbc7859c4c05bd53e58149bff499a Author: Reynold Xin Date: 2017-06-14T07:19:13Z [SPARK-21092][SQL] Wire SQLConf in logical plan and expressions commit b03af7ffe0650811aeaa376b639ac16a17e8c4a8 Author: Reynold Xin Date: 2017-06-14T07:22:56Z More docs commit ea091643ae48183b2a9294a6bc37ee3b991c9226 Author: Reynold Xin Date: 2017-06-14T21:35:25Z Merge with master commit 14f2b41642cb9adfc04a5aa95a0d0fc231205598 Author: Reynold Xin Date: 2017-06-14T23:25:19Z Update PruneFilter rule. commit a032106b8c98bd8a0e823d6576114dbdcb6da032 Author: Reynold Xin Date: 2017-06-14T23:28:30Z Properly unset configs. commit cec78b5cf1ced8322c8cd8e599a3197c50ed49c0 Author: Reynold Xin Date: 2017-06-15T01:50:07Z Update OuterJoinEliminationSuite commit 40de35c3ac6bdbf2bd2c43b9deca04a5cbdbc4ef Author: Reynold Xin Date: 2017-06-15T02:08:56Z [SPARK-21103][SQL] QueryPlanConstraints should be part of LogicalPlan --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #18310: [SPARK-21103][SQL] QueryPlanConstraints should be...
Github user rxin closed the pull request at: https://github.com/apache/spark/pull/18310 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #18310: [SPARK-21103][SQL] QueryPlanConstraints should be...
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/18310 [SPARK-21103][SQL] QueryPlanConstraints should be part of LogicalPlan ## What changes were proposed in this pull request? QueryPlanConstraints should be part of LogicalPlan, rather than QueryPlan, since the constraint framework is only used for query plan rewriting and not for physical planning. ## How was this patch tested? Should be covered by existing tests, since it is a simple refactoring. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark SPARK-21103 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/18310.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #18310 commit 90cff62eaffbbc7859c4c05bd53e58149bff499a Author: Reynold Xin Date: 2017-06-14T07:19:13Z [SPARK-21092][SQL] Wire SQLConf in logical plan and expressions commit b03af7ffe0650811aeaa376b639ac16a17e8c4a8 Author: Reynold Xin Date: 2017-06-14T07:22:56Z More docs commit ea091643ae48183b2a9294a6bc37ee3b991c9226 Author: Reynold Xin Date: 2017-06-14T21:35:25Z Merge with master commit 14f2b41642cb9adfc04a5aa95a0d0fc231205598 Author: Reynold Xin Date: 2017-06-14T23:25:19Z Update PruneFilter rule. commit a032106b8c98bd8a0e823d6576114dbdcb6da032 Author: Reynold Xin Date: 2017-06-14T23:28:30Z Properly unset configs. commit cec78b5cf1ced8322c8cd8e599a3197c50ed49c0 Author: Reynold Xin Date: 2017-06-15T01:50:07Z Update OuterJoinEliminationSuite commit 40de35c3ac6bdbf2bd2c43b9deca04a5cbdbc4ef Author: Reynold Xin Date: 2017-06-15T02:08:56Z [SPARK-21103][SQL] QueryPlanConstraints should be part of LogicalPlan --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org