[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-18 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-220208132 Thanks! The major reason is the existing users could still follow the previous behaviors. They assume we will accept such inputs. I will change the

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-18 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-220203561 @gatorsmile For 2.0, we will use `spark.sql.warehouse.dir` instead of using `hive.metastore.warehouse.dir` (the support of this has been dropped). --- If your project

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-14 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-219204110 In the PR https://github.com/apache/spark/pull/12812, we mention we will not use `hive.metastore.warehouse.dir` to set the location. Thus, I think we should issue

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-14 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-219203243 @yhuai @andrewor14 @rxin @liancheng **Question**: This PR is to set `spark.sql.warehouse.dir` by using the user-specified value of

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-219200482 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

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-219200467 **[Test build #58601 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58601/consoleFull)** for PR 13111 at commit

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

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

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-219200094 **[Test build #58601 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58601/consoleFull)** for PR 13111 at commit

[GitHub] spark pull request: [SPARK-15320] [SQL] Spark-SQL Cli Ignores Para...

2016-05-13 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/13111 [SPARK-15320] [SQL] Spark-SQL Cli Ignores Parameter hive.metastore.warehouse.dir What changes were proposed in this pull request? Although overriding `hive.metastore.warehouse.dir` in