[ 
https://issues.apache.org/jira/browse/FLINK-19386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17204438#comment-17204438
 ] 

Jark Wu commented on FLINK-19386:
---------------------------------

Hi [~hailong wang], thank you for sharing the thoughts. However, removing 
legacy planner dependency is not possible unless we have removed the legacy 
planner. So I would suggest to keep them for now. Remvoving lagecy planner is 
on the planner, but we need to solve some blocker problems, e.g. FLIP-129, 
FLIP-134, FLIP-140, FLIP-136, etc.. 

>From the issue description, I thought you just want to replace 
>{{JavaScalaConversionUtil}}. I would +1 for that. But I don't think it's time 
>to remove dependencies for connectors and formats. 

> Remove legacy table planner dependecy from connectors and formats
> -----------------------------------------------------------------
>
>                 Key: FLINK-19386
>                 URL: https://issues.apache.org/jira/browse/FLINK-19386
>             Project: Flink
>          Issue Type: Improvement
>          Components: Table SQL / Legacy Planner
>    Affects Versions: 1.11.0
>            Reporter: hailong wang
>            Assignee: hailong wang
>            Priority: Major
>
> I found some connectors denpendecy  legacy table planner and used in test 
> class.
> For example `HiveCatalogUseBlinkITCase `  use `JavaScalaConversionUtil` in 
> legacy table planner not blink planner. For FLIP-32 has be accepted, I think 
> we can remove dependecy of legacy table planner from connectors and formats.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to