[FLINK-3728] Throw meaningful exceptions for unsupported SQL features

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Implemented
• Affects Version/s: 1.1.0
• Fix Version/s: 1.1.0
• Component/s: Table SQL / API
• Assignee: Fabian Hueske
• Reporter: Vasia Kalavri
• Created: 11/Apr/16 13:47
• Updated: 24/May/16 09:38
• Resolved: 24/May/16 09:38

Description

We must explicitly exclude unsupported SQL features such as Grouping Sets from being translated to Flink programs.
Otherwise, the resulting program will compute invalid results.

For that we must restrict the Calcite rules that translate Logical RelNodes into DataSetRel or DataStreamRel nodes.
We may only translate to DataSetRel or DataStreamRel nodes if these support the semantics of the RelNode.

Not translating a RelNode will yield a Calcite CannotPlanException that we should catch and enrich with a meaningful error message.

Issue Links

• links to: GitHub Pull Request #2018