[
https://issues.apache.org/jira/browse/FLINK-13266?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16885156#comment-16885156
]
Timo Walther commented on FLINK-13266:
--------------------------------------
I would not use {{org.apache.flink.table.blink}} in any package name. According
to FLIP-32, {{flink-table-planner-blink}} will be renamed to
{{flink-table-planner}} and {{flink-table-runtime}} at some point. So the
package names should already represent this in order to not cause more merge
conflicts in the future. How about {{org.apache.flink.table.planner}} and
{{org.apache.flink.table.runtime}}?
> Relocate blink planner classes to avoid class clashes
> -----------------------------------------------------
>
> Key: FLINK-13266
> URL: https://issues.apache.org/jira/browse/FLINK-13266
> Project: Flink
> Issue Type: Sub-task
> Components: Table SQL / Planner
> Affects Versions: 1.9.0
> Reporter: Jark Wu
> Assignee: Jark Wu
> Priority: Blocker
> Fix For: 1.9.0
>
>
> We should have a list to relocate classes in {{flink-table-planner-blink}}
> and {{flink-table-runtime-blink}} to avoid class clashes to make both
> planners available in a lib directory.
> Note that, not all the classes can/should be relocated. For examples: calcite
> classes, {{PlannerExpressionParserImpl}} and so on.
> The relocation package name is up to discussion. A dedicated path is
> {{org.apache.flink.table.blink}}.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
