[GitHub] spark pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219436922 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219436923 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58641/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219436798 **[Test build #58641 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58641/consoleFull)** for PR 13115 at commit [`e756a0a`](https://github.com/apache/spark/commit/e756a0a820c164920e31b935d6309ee85b3bd524). * This patch **fails Spark unit 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219427662 **[Test build #58641 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58641/consoleFull)** for PR 13115 at commit [`e756a0a`](https://github.com/apache/spark/commit/e756a0a820c164920e31b935d6309ee85b3bd524). --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219213787 (I think it would be nicer if the PR description is fill up.) --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219213070 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219213071 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58605/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219213068 **[Test build #58605 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58605/consoleFull)** for PR 13115 at commit [`3b5eb9b`](https://github.com/apache/spark/commit/3b5eb9bb6bdf7377144b9bdd6c97a9cd5f39d088). * This patch **fails to build**. * 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13115#issuecomment-219212931 **[Test build #58605 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58605/consoleFull)** for PR 13115 at commit [`3b5eb9b`](https://github.com/apache/spark/commit/3b5eb9bb6bdf7377144b9bdd6c97a9cd5f39d088). --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user KaiXinXiaoLei commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-219212923 @andrewor14 See https://github.com/apache/spark/pull/13115, --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user KaiXinXiaoLei closed the pull request at: https://github.com/apache/spark/pull/10900 --- 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: [SPARK-12492] Using spark-sql commond to run q...
GitHub user KaiXinXiaoLei opened a pull request: https://github.com/apache/spark/pull/13115 [SPARK-12492] Using spark-sql commond to run query, write the event of SparkListenerJobStart See https://github.com/apache/spark/pull/10900 ## What changes were proposed in this pull request? (Please fill in changes proposed in this fix) ## How was this patch tested? (Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests) (If this patch involves UI changes, please attach a screenshot; otherwise, remove this) hot; otherwise, remove this) You can merge this pull request into a Git repository by running: $ git pull https://github.com/KaiXinXiaoLei/spark sqlPage2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/13115.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 #13115 commit 3b5eb9bb6bdf7377144b9bdd6c97a9cd5f39d088 Author: KaiXinXiaoLei Date: 2016-05-14T10:17:30Z sql page --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-216711577 BY THE WAY this code is all moved into `QueryExecution.scala` now since we deleted `HiveContext`. It might be easier to just create a new patch altogether. (see #12588) --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-216711282 It's just wrapping the things right? Then this LGTM assuming you didn't change anything else. @KaiXinXiaoLei please rebase. --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-210493191 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55929/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-210493184 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-210492856 **[Test build #55929 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55929/consoleFull)** for PR 10900 at commit [`f778d02`](https://github.com/apache/spark/commit/f778d02da274397ac9cbd160f527ff85b401e3a1). * This patch **fails Spark unit 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-210476544 **[Test build #55929 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55929/consoleFull)** for PR 10900 at commit [`f778d02`](https://github.com/apache/spark/commit/f778d02da274397ac9cbd160f527ff85b401e3a1). --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-208926178 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-208926182 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55611/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-208926170 **[Test build #55611 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55611/consoleFull)** for PR 10900 at commit [`64080c7`](https://github.com/apache/spark/commit/64080c72a73c0c77d1d037bdf6c8adea6b80e299). * This patch **fails Scala style 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-208925742 **[Test build #55611 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55611/consoleFull)** for PR 10900 at commit [`64080c7`](https://github.com/apache/spark/commit/64080c72a73c0c77d1d037bdf6c8adea6b80e299). --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-208455143 > @zsxwing I change it, Thanks. Did you push? --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user KaiXinXiaoLei commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-208319014 @zsxwing I change it, 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-207596874 Okey, I found the issue. `executedPlan` is a lazy val. When it's called at the first time, it may call `stringResult`. So if `executedPlan` is moved into `withNewExecutionId`, then the initialization of `withNewExecutionId` will be done in `withNewExecutionId` (this method will use `withNewExecutionId`), and we will suffer from nested execution id issue. @KaiXinXiaoLei could you update this PR to move `executedPlan` out of `withNewExecutionId` in order to initialize `executedPlan` outside `withNewExecutionId`? --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204186561 @andrewor14 Looks executedPlan must be called before calling `SQLExecution.withNewExecutionId`. Do you know why? --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204165558 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204165563 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54668/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204165308 **[Test build #54668 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54668/consoleFull)** for PR 10900 at commit [`d0e2693`](https://github.com/apache/spark/commit/d0e26930353a477717264d86a5036d8dc369bf3d). * This patch **fails Spark unit 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204149427 **[Test build #54668 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54668/consoleFull)** for PR 10900 at commit [`d0e2693`](https://github.com/apache/spark/commit/d0e26930353a477717264d86a5036d8dc369bf3d). --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204148427 LGTM pending tests --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-204147906 retest 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-203484299 **[Test build #54521 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54521/consoleFull)** for PR 10900 at commit [`d0e2693`](https://github.com/apache/spark/commit/d0e26930353a477717264d86a5036d8dc369bf3d). * This patch **fails Spark unit 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-203484553 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54521/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-203484543 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-203450655 **[Test build #54521 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54521/consoleFull)** for PR 10900 at commit [`d0e2693`](https://github.com/apache/spark/commit/d0e26930353a477717264d86a5036d8dc369bf3d). --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user KaiXinXiaoLei commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-203328686 Ok, I will change. 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-203038434 also cc @zsxwing --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10900#discussion_r57774278
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala ---
@@ -615,11 +615,13 @@ class HiveContext private[hive](
command.executeCollect().map(_.getString(0))
case other =>
-val result: Seq[Seq[Any]] =
other.executeCollectPublic().map(_.toSeq).toSeq
-// We need the types so we can output struct field names
-val types = analyzed.output.map(_.dataType)
-// Reformat to match hive tab delimited output.
-
result.map(_.zip(types).map(HiveContext.toHiveString)).map(_.mkString("\t")).toSeq
+SQLExecution.withNewExecutionId(other.sqlContext, this) {
--- End diff --
this should probably take in `self` instead of `other.sqlCotnext` just to
be safe
---
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: [SPARK-12492] Using spark-sql commond to run q...
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/10900#discussion_r57774355 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -615,11 +615,13 @@ class HiveContext private[hive]( command.executeCollect().map(_.getString(0)) --- End diff -- don't we need to do it here as well? More generally should we wrap it around the whole `stringResult`? --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user KaiXinXiaoLei commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-200819923 @JoshRosen Can u check? 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-174582176 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49994/ 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-174581473 **[Test build #49994 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49994/consoleFull)** for PR 10900 at commit [`7850d6b`](https://github.com/apache/spark/commit/7850d6b2f33969c349fad3a0657fb8c5f4e270c8). * 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 pull request: [SPARK-12492] Using spark-sql commond to run q...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-174582170 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 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: [SPARK-12492] Using spark-sql commond to run q...
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-174570726 Can you write a description for this PR to explain the motivation for this change? --- 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: [SPARK-12492] Using spark-sql commond to run q...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10900#issuecomment-174539117 **[Test build #49994 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49994/consoleFull)** for PR 10900 at commit [`7850d6b`](https://github.com/apache/spark/commit/7850d6b2f33969c349fad3a0657fb8c5f4e270c8). --- 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: [SPARK-12492] Using spark-sql commond to run q...
GitHub user KaiXinXiaoLei opened a pull request: https://github.com/apache/spark/pull/10900 [SPARK-12492] Using spark-sql commond to run query, write the event of SparkListenerJobStart You can merge this pull request into a Git repository by running: $ git pull https://github.com/KaiXinXiaoLei/spark sqlPage Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/10900.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 #10900 commit 7850d6b2f33969c349fad3a0657fb8c5f4e270c8 Author: KaiXinXiaoLei Date: 2016-01-25T14:51:49Z add event --- 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
