Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15054
---
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 ena
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79540609
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalogSuite.scala
---
@@ -444,7 +444,7 @@ class SessionCatalogSuite ex
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79540494
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -282,6 +271,24 @@ class SessionCatalog(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79539702
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -357,6 +346,21 @@ class SessionCatalog(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79528505
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -357,6 +346,21 @@ class SessionCatalog(
t
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79478675
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -37,7 +38,9 @@ case class AnalyzeTableCommand(t
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79478609
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -246,30 +246,42 @@ class SessionCatalog(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79344601
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -246,30 +246,42 @@ class SessionCatalog(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79340704
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -37,7 +38,9 @@ case class AnalyzeTableCommand(ta
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79340506
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -246,30 +246,42 @@ class SessionCatalog(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79320015
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -65,7 +64,11 @@ case class CreateTableLikeCommand(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79281274
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -65,7 +64,11 @@ case class CreateTableLikeCommand(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79275910
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -360,6 +360,7 @@ trait CheckAnalysis extends Pred
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r79056087
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -360,6 +360,7 @@ trait CheckAnalysis extends Predic
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78900278
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -360,6 +360,7 @@ trait CheckAnalysis extends Predi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78803231
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
---
@@ -95,15 +155,38 @@ class SQLViewSuite extends QueryTest wi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78802228
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -913,7 +913,7 @@ class DDLSuite extends QueryTest with Sha
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78802202
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
---
@@ -95,15 +155,38 @@ class SQLViewSuite extends QueryTest wi
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78792855
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala
---
@@ -95,15 +155,38 @@ class SQLViewSuite extends QueryTest wit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78792645
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -913,7 +913,7 @@ class DDLSuite extends QueryTest with Shar
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78784318
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -91,6 +91,9 @@ case class AnalyzeTableCommand(t
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78765976
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -91,6 +91,9 @@ case class AnalyzeTableCommand(ta
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15054#discussion_r78766111
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala ---
@@ -266,6 +267,12 @@ case class AlterTableUnsetPropertiesCommand(
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/15054
[SPARK-17502] [SQL] Fix Multiple Bugs in DDL Statements on Temporary Views
[WIP]
### What changes were proposed in this pull request?
- When the permanent tables/views do not exist but the t
24 matches
Mail list logo