[BEAM-2679] SQL DSL API Review

• Type: Task
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: 2.2.0
• Component/s: dsl-sql
• Labels: Done dsl_sql_merge
• Assignee: Tyler Akidau
• Reporter: Mingmin Xu
• Created: 25/Jul/17 18:10
• Updated: 16/May/20 13:39
• Resolved: 24/Oct/17 16:29

Description

7. API Review: This will be a large addition to the API. We will need to have a holistic API review of the proposed additions before merge. Note: this may result in large changes to the API surface. Committers involved will do their best to prevent this as things evolve, but it is always a possibility.

Sub-Tasks

• update JavaDoc of BeamSql: Resolved
• accept SerializableFunction as UDF: Resolved
• accept CombineFn as UDAF: Resolved
• change visibility of BeamSqlTable: Resolved
• add getFieldTypeByIndex(index) in BeamRecordSqlType: Resolved
• rename size() in BeamRecordType: Resolved
• rename size() in BeamRecord: Resolved
• Switch from primitives to Java types: Resolved
• update JavaDoc for BeamRecord: Resolved
• update exposed methods in BeamSqlEnv: Resolved
• improve JavaDoc in BeamSqlEnv: Resolved
• make BeamRecord an immutable type: Resolved
• remove nullFields in BeamRecord: Resolved
• Remove redundant windowing information from the BeamRecord itself: Resolved
• update BeamSqlExample: Resolved
• rename simpleQuery to query and query to queryMulti: Resolved
• list supported SQL grammar in BNF: Resolved
• update page dsl_sql: Resolved
• update JavaDoc of BeamSortRel: Resolved