wuchong 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-581095203
 
 
   Thanks @JingsongLi , according to the updated description. I'm in favor of 
option#1 which passes `FlinkPlannerImpl` via context, because it is a true 
fixing not a temporary workaround and gives us the ability to extend features 
in the future.  Btw, please try to expose a small interface, rather than a 
whole `FlinkPlannerImpl`.

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