twalthr commented on a change in pull request #8404: [FLINK-11476][table]
Create CatalogManager to manage multiple catalogs
URL: https://github.com/apache/flink/pull/8404#discussion_r283654823
##########
File path:
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/TableEnvironment.java
##########
@@ -119,8 +148,13 @@
/**
* Scans a registered table and returns the resulting {@link Table}.
*
- * <p>A table to scan must be registered in the TableEnvironment. It
can be either directly
- * registered or be a member of an {@link ExternalCatalog}.
+ * <p>A table to scan must be registered in the {@link
TableEnvironment}. It can be either directly
+ * registered or be a member of an {@link ExternalCatalog} or {@link
Catalog}.
+ *
+ * <p>First we try to look for {@code [default-path].[table-path]} if
no object is found we assume the
Review comment:
add an example here. How can a `default-path` and `table-path`look like? And
you mention object path and object, I think this is unrelated in this method.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
