cloud-fan commented on a change in pull request #25771: [SPARK-28970][SQL]
Implement USE CATALOG/NAMESPACE for Data Source V2
URL: https://github.com/apache/spark/pull/25771#discussion_r325488148
##########
File path:
sql/core/src/test/scala/org/apache/spark/sql/connector/DataSourceV2SQLSuite.scala
##########
@@ -780,11 +794,8 @@ class DataSourceV2SQLSuite
test("ShowNamespaces: default v2 catalog is not set") {
spark.sql("CREATE TABLE testcat.ns.table (id bigint) USING foo")
- val exception = intercept[AnalysisException] {
- sql("SHOW NAMESPACES")
- }
-
- assert(exception.getMessage.contains("No default v2 catalog is set"))
+ // The current catalog is resolved to a v2 session catalog.
+ testShowNamespaces("SHOW NAMESPACES", Seq())
Review comment:
wait, I think the previous conclusion was to ignore current name space and
always list the root namespaces for `SHOW NAMESPACES`. cc @rdblue
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
