[FLINK-13168] clarify isBatch/isStreaming/isBounded flag in flink planner and blink planner

• Type: Improvement
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 1.9.0
• Component/s: Table SQL / Legacy Planner, Table SQL / Planner
• Labels: pull-request-available
• Assignee: godfrey he
• Reporter: godfrey he
• Created: 09/Jul/19 10:03
• Updated: 18/Jul/19 16:05
• Resolved: 17/Jul/19 11:52

Description

in blink planner & flink planner, there are many isBatch and isStreaming flags, they have different meaning in different place. which makes reader and coder crazy. especially in blink planner, Only StreamTableSource could be used for both batch and stream. is bounded StreamTableSource means batch, unbounded means stream ?

we should make it clear:
1. isBatch in ConnectorCatalogTable, which tells if the tableSource/tableSink is BatchTableSource/BatchTableSink
2. isBatch in TableSourceQueryOperation, which tells if the tableSource is BatchTableSource
3. isStreaming in TableSourceTable, which tells if if the current table is on stream planner
4. bounded StreamTableSource could be used for both batch and stream, while unbounded StreamTableSource could only be used for stream.

Issue Links

• causes: FLINK-13327 Blink planner not compiling with Scala 2.12
• links to: GitHub Pull Request #9035