Ignite 2.3 still not support group_concat

[james wu <ja...@coupang.com>]

The new ignite 2.3 release doc said there are aggregation group_concat and
related usage
https://apacheignite-sql.readme.io/docs/group_concat, but look into the
source code, this function still not support
/** {@inheritDoc} */
    @Override public String getSQL() {
        String text;

        switch (type) {
            case GROUP_CONCAT:
                throw new UnsupportedOperationException();

            case COUNT_ALL:
                return "COUNT(*)";

            default:
                text = type.name();

                break;
        }

        if (distinct)
            return text + "(DISTINCT " + child().getSQL() + ")";

        return text + StringUtils.enclose(child().getSQL());
    }



--
Sent from: http://apache-ignite-users.70518.x6.nabble.com/

Re: Ignite 2.3 still not support group_concat

[Evgenii Zhuravlev <e....@gmail.com>]

Hi James,

Nice catch! Yes, it's not supported yet. Looks like it was mistakenly added
to the documentation in version 2.3. Just suggested deleting this page from
the doc. It should be removed soon.

Regards,
Evgenii

2017-11-02 4:53 GMT+03:00 james wu <ja...@coupang.com>:

> The new ignite 2.3 release doc said there are aggregation group_concat and
> related usage
> https://apacheignite-sql.readme.io/docs/group_concat, but look into the
> source code, this function still not support
> /** {@inheritDoc} */
>     @Override public String getSQL() {
>         String text;
>
>         switch (type) {
>             case GROUP_CONCAT:
>                 throw new UnsupportedOperationException();
>
>             case COUNT_ALL:
>                 return "COUNT(*)";
>
>             default:
>                 text = type.name();
>
>                 break;
>         }
>
>         if (distinct)
>             return text + "(DISTINCT " + child().getSQL() + ")";
>
>         return text + StringUtils.enclose(child().getSQL());
>     }
>
>
>
> --
> Sent from: http://apache-ignite-users.70518.x6.nabble.com/
>