aokolnychyi commented on code in PR #57799:
URL: https://github.com/apache/spark/pull/57799#discussion_r3770840103
##########
sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/TableCatalog.java:
##########
@@ -196,12 +196,35 @@ default Table loadTable(Identifier ident, long timestamp)
throws NoSuchTableExce
}
/**
- * Load table metadata by {@link Identifier identifier} from the catalog,
forwarding all
- * user-specified options.
+ * Returns the raw option keys that may affect the table state selected by
Review Comment:
How about restructuring the first paragraph a bit?
```
/**
* Returns the connector-specific option keys that select the table state
(such as a branch, tag,
* snapshot, or version) and therefore must be known when the table is
loaded. Keys that Spark
* parses and handles itself, such as time travel, must not be listed here.
* <p>
```
--
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]
