[GitHub] flink issue #5441: [FLINK-8607] [table] Add a basic embedded SQL CLI client

2018-03-08 Thread twalthr 
Github user twalthr commented on the issue:

https://github.com/apache/flink/pull/5441
  
Hi @hzyuemeng1, in your terms we translate this as "sql -> table api -> 
runtime". But actually it is more like "sql api with calcite -> runtime". 
In the long-term view we will further optimize the "sql api with calcite" 
part.


---

[GitHub] flink issue #5441: [FLINK-8607] [table] Add a basic embedded SQL CLI client

2018-03-08 Thread hzyuemeng1 
Github user hzyuemeng1 commented on the issue:

https://github.com/apache/flink/pull/5441
  
@twalthr ,do we implement this base on table api such as sql -> table api 
-> runtime, or new plan:sql -> calcite->whole stage codegen -runtime.


---

[GitHub] flink issue #5441: [FLINK-8607] [table] Add a basic embedded SQL CLI client

2018-02-17 Thread twalthr 
Github user twalthr commented on the issue:

https://github.com/apache/flink/pull/5441
  
Thank you @fhueske. I will merge this now and open some follow-up issues.


---