[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/17340#discussion_r106780117 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -732,40 +676,6 @@ abstract class

[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/17340#discussion_r106780097 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -196,50 +196,6 @@ abstract class

[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/17340#discussion_r106780090 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -196,50 +196,6 @@ abstract class

[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/17340#discussion_r106780038 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -196,50 +196,6 @@ abstract class

[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/17340#discussion_r106780033 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -196,50 +196,6 @@ abstract class

[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/17340#discussion_r106780030 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -196,50 +196,6 @@ abstract class

[GitHub] spark pull request #17340: [SPARK-20013][SQL] merge renameTable to alterTabl...

2017-03-18 Thread windpiger
GitHub user windpiger opened a pull request: https://github.com/apache/spark/pull/17340 [SPARK-20013][SQL] merge renameTable to alterTable in ExternCatalog ## What changes were proposed in this pull request? ## How was this patch tested? delete some tests in