[GitHub] spark pull request: [Minor][SQL] Replace logger name
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/10555 [Minor][SQL] Replace logger name This is very minor update. @rxin Please see if it is proper for you. If not, I will close it. Thanks. You can merge this pull request into a Git repository by running: $ git pull https://github.com/viirya/spark-1 minor-parser-logger Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/10555.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 #10555 commit 2132171c7e0137bab78082857c40a431de4ab039 Author: Liang-Chi HsiehDate: 2016-01-02T14:44:05Z Minor: replace logger name. --- 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: [Minor][SQL] Replace logger name
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168396939 LGTM - missed that one. --- 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: [Minor][SQL] Replace logger name
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168405571 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48585/ 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: [Minor][SQL] Replace logger name
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168405569 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: [Minor][SQL] Replace logger name
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168397196 **[Test build #48585 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48585/consoleFull)** for PR 10555 at commit [`2132171`](https://github.com/apache/spark/commit/2132171c7e0137bab78082857c40a431de4ab039). --- 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: [Minor][SQL] Replace logger name
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168399954 @viirya I am working on a PR to remove all Hive dependencies from the parser. In order to do this, I am also touching the ```ParseDriver```. That might supercede this 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 pull request: [Minor][SQL] Replace logger name
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168405533 **[Test build #48585 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48585/consoleFull)** for PR 10555 at commit [`2132171`](https://github.com/apache/spark/commit/2132171c7e0137bab78082857c40a431de4ab039). * 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: [Minor][SQL] Replace logger name
Github user viirya closed the pull request at: https://github.com/apache/spark/pull/10555 --- 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: [Minor][SQL] Replace logger name
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10555#issuecomment-168401446 Ok. No problem. Close this now. --- 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