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: [email protected] With regards, Apache Git Services
