JingsongLi commented on issue #10989: [FLINK-15840][table-planner-blink] 
ClassCastException is thrown when use tEnv.from for temp/catalog table
URL: https://github.com/apache/flink/pull/10989#issuecomment-581090904
 
 
   Hi @wuchong , you can take a look to my solution 1, I think that is what are 
you mean. We don't need `FlinkToRelContext` any more, we can get 
`FlinkPlannerImpl` directly from `FlinkContext`. As I said, I think 
`FlinkToRelContext` is a bad way.
   
   I wanted to do that way.
   
   But discussed with @danny0405 offline:
   - Support computed column and watermark(I don't think there are some 
difference between computed column and watermark) in table api is not the goal 
of 1.10.
   - "Separate the computed column translation to parser stage" is his plan to 
do in 1.11.

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

Reply via email to