[GitHub] spark pull request: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread liancheng
Github user liancheng closed the pull request at: https://github.com/apache/spark/pull/5012 --- 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: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/5012#issuecomment-79518220 @srowen Adding `sql.functions` is more convenient, I'd prefer #5011 over this PR. Thanks for pointing this out! --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5012#issuecomment-79035346 [Test build #28564 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28564/consoleFull) for PR 5012 at commit

[GitHub] spark pull request: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5012#issuecomment-79035412 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5012#issuecomment-78982948 [Test build #28564 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28564/consoleFull) for PR 5012 at commit

[GitHub] spark pull request: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/5012#issuecomment-78982053 @liancheng see also https://github.com/apache/spark/pull/5011 just submitted. Is the new import vinod added needed? --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SQL] [Build] [Minor] Removes nonexistent sql....

2015-03-13 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/5012 [SQL] [Build] [Minor] Removes nonexistent sql.Dsl imports from sql/console and hive/console The `org.apache.spark.sql.Dsl` object has been removed since branch-1.3, should also remove them from