KurtYoung commented on issue #8844: [FLINK-12951][table-planner] Add logic to 
bridge DDL to table source(…
URL: https://github.com/apache/flink/pull/8844#issuecomment-510290992
 
 
   > How about supporting DDL & DML both in `sqlUpdate`? This can avoid to 
introduce a new `sql()` interface right now.
   > 
   > JDBC has a similar design about this, the `java.sql.Statement` interface 
has [`ResultSet executeQuery(String 
sql)`](https://docs.oracle.com/javase/8/docs/api/java/sql/Statement.html#executeQuery-java.lang.String-)
 method to execute only SELECT statement and return a query result (aka. 
Table), and also has a [`int executeUpdate(String 
sql)`](https://docs.oracle.com/javase/8/docs/api/java/sql/Statement.html#executeUpdate-java.lang.String-)
 method to execute DMLs and DDLs and returns effected rows.
   > 
   > So I think it makes sense to support DDL in `sqlUpdate` because it is also 
an update statement.
   
   Sounds like a reasonable solution

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

Reply via email to