amaliujia commented on code in PR #47684:
URL: https://github.com/apache/spark/pull/47684#discussion_r1711876566
##########
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:
Do we need to also document non true case?
Is non true value invalid, or it is valid but ignored, etc.?
--
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]
