[GitHub] [flink] dawidwys commented on issue #8859: [FLINK-12905][table-planner] Enable querying CatalogViews in legacy planner

[GitBox <gi...@apache.org>]

dawidwys commented on issue #8859: [FLINK-12905][table-planner] Enable querying CatalogViews in legacy planner
URL: https://github.com/apache/flink/pull/8859#issuecomment-558045818
 
 
   After some discussions with @twalthr over @lirui-apache comment I think we should not limit the support of views to the ones that have a corresponding dialect to the one of SqlParser. In a longer run we should have a SqlDialect flag in a View definition, but it should be a self contained information independent of the configuration of the SqlParser. (E.g. add Hive specific operators when translating the view to a relational tree, reconfiguring/creating a parser compatible with the view etc.).
   
   I reverted the PR to the state where we perform only best effort parsing of the view.

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


With regards,
Apache Git Services