[GitHub] spark issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 @cloud-fan @gatorsmile @davies @rxin @hvanhovell Thank you all. This was my first PR! --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13418 thanks, merging to master and 2.0! --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13418 **[Test build #3045 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3045/consoleFull)** for PR 13418 at commit [`77166bc`](https://github.com/apache/spark/commit/77166bc5a6989382d628ba293fd9751c2ca3e20a). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/13418 whoops triggered a build on this one... sorry about that --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13418 **[Test build #3045 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3045/consoleFull)** for PR 13418 at commit [`77166bc`](https://github.com/apache/spark/commit/77166bc5a6989382d628ba293fd9751c2ca3e20a). --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 @cloud-fan Thank you. --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13418 LGTM pending jenkins --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 Thank you @cloud-fan. I mentioned the local relations in the test case description and move the test cases under withTempTable. --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 @cloud-fan I moved the unit tests to a new test case. Thank you. --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13418 LGTM except one test comment. --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 @cloud-fan I replaced p.expressions with projectList. Thanks. --- 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 issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 @gatorsmile @davies @rxin @cloud-fan I've incorporated the comments. Please advise. Thank you. --- 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