slinkydeveloper commented on code in PR #18487:
URL: https://github.com/apache/flink/pull/18487#discussion_r849261310


##########
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/internal/TableEnvironmentImpl.java:
##########
@@ -1899,4 +1905,14 @@ public TableResult executeJsonPlan(String jsonPlan) {
         }
         return executeInternal(transformations, sinkIdentifierNames);
     }
+
+    public String[] showConnectors() {
+        return FactoryUtil.discoverFactories(userClassLoader).stream()
+                .filter(
+                        t ->
+                                t instanceof DynamicTableSourceFactory
+                                        || t instanceof 
DynamicTableSinkFactory)

Review Comment:
   not sure. TBH I was venting the idea somewhere else to stop adding these 
`SHOW` special syntax, as I'm not really a fan of these as new syntax is always 
harder to mantain, but rather go with a proper information_schema kind of 
thing, which could include also something to show connectors, formats, etc... 
Feel free to join the discussion there 
https://issues.apache.org/jira/browse/FLINK-25684



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

Reply via email to