davidradl commented on code in PR #25810:
URL: https://github.com/apache/flink/pull/25810#discussion_r1893689041
##########
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/TableEnvironment.java:
##########
@@ -1040,6 +1066,32 @@ void createTemporarySystemFunction(
*/
boolean dropTemporaryView(String path);
+ /**
+ * Drops a view registered in the given path.
+ *
+ * <p>Temporary objects can shadow permanent ones. If a temporary object
exists in a given path,
+ * make sure to drop the temporary object first using {@link
#dropTemporaryView}. This method
Review Comment:
nit: I would move sentence "This method can only drop permanent objects" to
the beginning of the paragraph. So it is obvious what this method is before we
talk about temporary objects.
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
