Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/16674#discussion_r100593400
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
 ---
    @@ -624,6 +624,9 @@ class Analyzer(
           } catch {
             case _: NoSuchTableException =>
               u.failAnalysis(s"Table or view not found: ${u.tableName}")
    +        // If the database is not valid, also throw an AnalysisException.
    +        case _: NoSuchDatabaseException =>
    +          u.failAnalysis(s"Table or view not found: ${u.tableName}")
    --- End diff --
    
    shall we use the qualified name? or users are hard to find out that the 
database doesn't exist, given only the table name.


---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to