[FLINK-3489] Refactor Table API before merging into master

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.1.0
• Component/s: Table SQL / API
• Assignee: Vasia Kalavri
• Reporter: Timo Walther
• Created: 24/Feb/16 08:59
• Updated: 15/Mar/16 18:15
• Resolved: 15/Mar/16 18:15

Description

Before we merge the `tableOnCalcite` branch back into master, we should do some package/class refactoring. Here are some ideas:

• Move class TreeNode into expressions or its methods directly into Expression.
• Remove the org.apache.flink.api.table.trees package.
• Rename org.apache.flink.api.table.typeinfo to typeutils to be in sync with other Flink modules.
• Move TypeConverter to typeutils
• Remove ExpressionException it is currently only used by typeInfo() of Expression
• Remove typeInfo() of Expression and subclasses. It is not used.
• Add ExecutionEnvironment as parameter of TableEnvironment (necessary for Table API specific input formats) in order to not break the API in future?
• Remove println in translators.

Feel free to propose other changes.