[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-22 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r675051098 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3613,110 +3656,20 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-22 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r675048044 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/v2Commands.scala ## @@ -1099,21 +1066,83 @@ case class

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-21 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r674350772 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,18 +3568,67 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-21 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r674348960 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-21 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r674150375 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-21 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r674054248 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-20 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r673295251 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala ## @@ -1025,17 +938,23 @@ trait CheckAnalysis

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-20 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r673291583 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala ## @@ -1025,17 +938,23 @@ trait CheckAnalysis

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-20 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r673289064 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala ## @@ -1025,17 +938,23 @@ trait CheckAnalysis

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-20 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r672486685 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-20 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r672486685 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-19 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r672489238 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-19 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r672486685 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -3574,15 +3568,64 @@ class Analyzer(override val

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-17 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r671595283 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statements.scala ## @@ -229,22 +228,13 @@ case class

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-16 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r671595283 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statements.scala ## @@ -229,22 +228,13 @@ case class

[GitHub] [spark] imback82 commented on a change in pull request #33200: [SPARK-36006][SQL] Migrate ALTER TABLE ... ADD/REPLACE COLUMNS commands to use UnresolvedTable to resolve the identifier

2021-07-16 Thread GitBox
imback82 commented on a change in pull request #33200: URL: https://github.com/apache/spark/pull/33200#discussion_r671593276 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/v2Commands.scala ## @@ -1101,19 +1068,80 @@ case class