You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Julian Hyde (JIRA)" <ji...@apache.org> on 2019/01/17 21:58:00 UTC
[jira] [Commented] (CALCITE-2786) Add order by clause support for
JSON_ARRAYAGG
[ https://issues.apache.org/jira/browse/CALCITE-2786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16745548#comment-16745548 ]
Julian Hyde commented on CALCITE-2786:
--------------------------------------
I haven't reviewed your change, but I agree with your analysis that ORDER BY inside JSON_ARRAYAGG should be treated as (and implemented as) syntactic sugar for WITHIN GROUP. And we should allow both.
> Add order by clause support for JSON_ARRAYAGG
> ---------------------------------------------
>
> Key: CALCITE-2786
> URL: https://issues.apache.org/jira/browse/CALCITE-2786
> Project: Calcite
> Issue Type: Improvement
> Components: core
> Reporter: Hongze Zhang
> Assignee: Hongze Zhang
> Priority: Major
>
> Both Oracle[1] and SQL:2016[2] supports or introduces an optional order by clause in JSON_ARRAYAGG, the feature can be implemented since Calcite already supports sorted aggregation.
> AFAICS, JSON_ARRAYAGG's order by clause has no difference with the definition of WITHIN GROUP's, so I think it might be feasible to do sort by expanding JSON_ARRAYAGG(... ORDER BY ...) to JSON_ARRAYAGG(...) WITHIN GROUP (ORDER BY...) during SQL parsing.
> [1] https://docs.oracle.com/en/database/oracle/oracle-database/12.2/sqlrf/JSON_ARRAYAGG.html#GUID-6D56077D-78DE-4CC0-9498-225DDC42E054
> [2] http://standards.iso.org/ittf/PubliclyAvailableStandards/c065143_ISO_IEC_TR_19075-5_2016.zip
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)