[GitHub] [flink] godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] Bump Calcite dependency to 1.20.0 in sql parser & flink planner

2019-07-10 Thread GitBox
godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] 
Bump Calcite dependency to 1.20.0 in sql parser & flink planner
URL: https://github.com/apache/flink/pull/9056#issuecomment-510304178
 
 
   > > > Oops.. I only had validated the dependency in flink-table module. I 
will re-validate it in whole project
   > > 
   > > 
   > > Maybe also find a way to verify python table api with blink planner.
   > 
   > ok, i will fix it
   
   @dianfu told me that flink-python does not support blink planner now


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] Bump Calcite dependency to 1.20.0 in sql parser & flink planner

2019-07-10 Thread GitBox
godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] 
Bump Calcite dependency to 1.20.0 in sql parser & flink planner
URL: https://github.com/apache/flink/pull/9056#issuecomment-510297306
 
 
   > > Oops.. I only had validated the dependency in flink-table module. I will 
re-validate it in whole project
   > 
   > Maybe also find a way to verify python table api with blink planner.
   
   ok, i will fix it


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] Bump Calcite dependency to 1.20.0 in sql parser & flink planner

2019-07-10 Thread GitBox
godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] 
Bump Calcite dependency to 1.20.0 in sql parser & flink planner
URL: https://github.com/apache/flink/pull/9056#issuecomment-510293697
 
 
   Oops.. I only had validated the dependency in flink-table module. I will 
re-validate it in whole project


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [flink] godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] Bump Calcite dependency to 1.20.0 in sql parser & flink planner

2019-07-09 Thread GitBox
godfreyhe commented on issue #9056: [FLINK-13185] [sql-parser][table-planner] 
Bump Calcite dependency to 1.20.0 in sql parser & flink planner
URL: https://github.com/apache/flink/pull/9056#issuecomment-509920350
 
 
   cc @danny0405 @walterddr 


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:
us...@infra.apache.org


With regards,
Apache Git Services