cloud-fan commented on code in PR #47684:
URL: https://github.com/apache/spark/pull/47684#discussion_r1713133043


##########
sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/TableCatalog.java:
##########
@@ -52,7 +52,8 @@ public interface TableCatalog extends CatalogPlugin {
 
   /**
    * A reserved property to indicate that the table location is managed, not 
user-specified.
-   * If this property is "true", SHOW CREATE TABLE will not generate the 
LOCATION clause.
+   * If this property is "true", it means it's a managed table even if it has 
a location. As an
+   * example, SHOW CREATE TABLE will not generate the LOCATION clause.

Review Comment:
   The non-true case is hard to document, as treating tables with location as 
external tables is a Hive-specific behavior.



-- 
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]

Reply via email to