[CALCITE-2670] Combine similar JSON aggregate functions in operator table

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.18.0
• Component/s: core
• Assignee: Julian Hyde
• Reporter: Hongze Zhang
• Created: 14/Nov/18 11:47
• Updated: 25/Feb/19 14:47
• Resolved: 06/Dec/18 08:57

Description

In current code master of Calcite, operator JSON_ARRAYAGG and JSON_OBJECTAGG are separated to JSON_ARRAYAGG_NULL_ON_NULL, JSON_ARRAYAGG_ABSENT_ON_NULL, JSON_OBJECTAGG_NULL_ON_NULL and JSON_OBJECTAGG_ABSENT_ON_NULL in SqlStdOperatorTable.java. This is OK for now, but may deliver more difficulty on extending syntax of JSON_ARRAYAGG and JSON_OBJECTAGG and on implementing logical plan by adapters.

This is basically to combine the 4 operators to 2, this is a improvement list:

1. Combine JSON_ARRAYAGG_NULL_ON_NULL and JSON_ARRAYAGG_ABSENT_ON_NULL;
2. Combine JSON_OBJECTAGG_NULL_ON_NULL and JSON_OBJECTAGG_ABSENT_ON_NULL;
3. SYMBOL type Support for JavaTypeFactoryImpl#getJavaClass This is to generate Enum java type for SYMBOL;
4. Add SQL-to-Rel test cases for JSON functions.

Issue Links

• is depended upon by: CALCITE-2575 Release Calcite 1.18.0
• links to: PR #916