amaliujia commented on code in PR #37287:
URL: https://github.com/apache/spark/pull/37287#discussion_r939283477
##########
sql/core/src/main/scala/org/apache/spark/sql/catalog/Catalog.scala:
##########
@@ -90,30 +92,31 @@ abstract class Catalog {
/**
* Returns a list of columns for the given table/view or temporary view.
*
- * @param tableName is either a qualified or unqualified name that
designates a table/view.
- * If no database identifier is provided, it refers to a
temporary view or
- * a table/view in the current database.
+ * @param tableName is either a qualified or unqualified name that
designates a table/view. It
+ * follows the same resolution rule with SQL: search for
temp views first then
+ * table/views in the current database (namespace).
* @since 2.0.0
*/
@throws[AnalysisException]("table does not exist")
def listColumns(tableName: String): Dataset[Column]
/**
- * Returns a list of columns for the given table/view in the specified
database.
+ * Returns a list of columns for the given table/view in the specified
database under the Hive
+ * Metastore.
Review Comment:
+1 this looks nice to explicitly says HMS
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
