[FLINK-15950] Remove registration of TableSource/TableSink in ConnectTableDescriptor

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: Table SQL / API
• Labels: breaking-api pull-request-available
• Assignee: Unassigned
• Reporter: Kurt Young
• Created: 07/Feb/20 10:19
• Updated: 11/Jun/20 15:23
• Resolved: 11/Jun/20 15:23

Description

This ticket would track the removal of direct TableSource/TableSink registration to TableEnvironment. Since we have lots of tests rely on this, I will create some sub tasks to divide this big one. 

Discuss thread: http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Remove-registration-of-TableSource-TableSink-in-Table-Env-and-ConnectTableDescriptor-td37270.html

 

update [2020/6/11]: Since we don't have a good alternative solution for registering user defined source/sink/factory yet, so we will defer the removal of registration in table env to next version.

Issue Links

• links to: GitHub Pull Request #12076

Sub-Tasks

• Remove register source and sink in ConnectTableDescriptor: Closed
• Remove register source and sink in test cases of blink planner: Closed
• Avoid register source in TableTestBase#addTableSource: Closed
• Schema#proctime and Schema#rowtime don't work in TableEnvironment#connect code path: Closed
• Reintroduce registerTableSource/Sink methods for table env: Closed
• Remove registration of TableSource/TableSink in Table Env: Closed