[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-25 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r64673144 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -354,7 +356,27 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-20 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r64126341 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -354,7 +356,27 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-20 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r64100525 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -354,7 +356,27 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-19 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63903975 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -354,7 +356,27 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-19 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63869408 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -354,7 +356,27 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/13120#issuecomment-220172124 @yhuai @liancheng I updated the code and also did some manual tests for creating table with a real hdfs path to one of my clusters. For example: ```scala>

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63772240 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63719953 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63718106 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63687429 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63686818 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-18 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63686590 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63605497 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -451,13 +462,15 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63605320 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63604166 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63603796 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -451,13 +462,15 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63599112 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-15 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63300902 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-15 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63298718 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13120#issuecomment-219253785 Can one of the admins verify this patch? --- 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-15269][SQL] Set provided path to Catalo...

2016-05-14 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/13120#issuecomment-219253654 cc @liancheng @yhuai @gatorsmile 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

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-14 Thread xwu0226
GitHub user xwu0226 opened a pull request: https://github.com/apache/spark/pull/13120 [SPARK-15269][SQL] Set provided path to CatalogTable.storage.locationURI when creating external non-hive compatible table ## What changes were proposed in this pull request? ### Symptom