danny0405 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-510404004
 
 
   Thanks @fhueske for your nice review comments, i also think we should be 
caution for adding a pretty new interface and also deprecate the old one(which 
are most commonly used).
   
   I definitely agree with your point that we should fire a fresh new 
discussion early on in the development phase of 1.10 about the relationship of 
`sql` `sqlUpdate` and `sqlQuery`, especially when we introduce other DDLs 
grammar like `Alter table` or `Show tables` etc.
   
   Broadly i agree the idea to put `CREATE TABLE` ddl in `sqlUpdate`, also i 
have fired another JIRA issue: https://issues.apache.org/jira/browse/FLINK-13209

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