Support GroupConcat sql for aggregating multiple shards(#33797)

[YaoFly]

Fixes #33797.

Changes proposed in this pull request:

• Support GroupConcat sql for aggregating multiple shards
• Support GroupConcat sql for default separator

---

Before committing this PR, I'm sure that I have checked the following options:

• My code follows the code of conduct of this project.
• I have self-reviewed the commit code.
• I have (or in comment I request) added corresponding labels for the pull request.
• I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
• I have made corresponding changes to the documentation.
• I have added corresponding unit tests for my changes.
• I have updated the Release Notes of the current development version. For more details, see Update Release Note

[YaoFly]

@strongduanmu PTAL. I don't think my changes will affect the "E2E - Operation / E2E - pipeline on postgres:12-alpine (pull_request) ",

[strongduanmu]

Hi @YaoFly, can you add e2e test in https://github.com/apache/shardingsphere/blob/master/test/e2e/sql/src/test/resources/cases/dql/e2e-dql-select-aggregate.xml

Another question is, besides MySQL database, are there any other databases that support GROUP_CONCAT? If so, please add them to the E2E Case.

[YaoFly]

Hi @YaoFly, can you add e2e test in https://github.com/apache/shardingsphere/blob/master/test/e2e/sql/src/test/resources/cases/dql/e2e-dql-select-aggregate.xml

Another question is, besides MySQL database, are there any other databases that support GROUP_CONCAT? If so, please add them to the E2E Case.

ok, I will

[YaoFly]

Hi @YaoFly, can you add e2e test in https://github.com/apache/shardingsphere/blob/master/test/e2e/sql/src/test/resources/cases/dql/e2e-dql-select-aggregate.xml

Another question is, besides MySQL database, are there any other databases that support GROUP_CONCAT? If so, please add them to the E2E Case.

e2e test added. besides MySQL database, other database required additional code development, which I suggest supporting them in other pull requests?

[strongduanmu]

Hi @YaoFly, can you add e2e test in https://github.com/apache/shardingsphere/blob/master/test/e2e/sql/src/test/resources/cases/dql/e2e-dql-select-aggregate.xml
Another question is, besides MySQL database, are there any other databases that support GROUP_CONCAT? If so, please add them to the E2E Case.

e2e test added. besides MySQL database, other database required additional code development, which I suggest supporting them in other pull requests?

OK, you are welcome to submit more PRs to support other database.

[strongduanmu]

Hi @YaoFly, thanks for everything you did on this PR, it still has some minor issues that need your help.

[strongduanmu]

@YaoFly In addition, you need to update this PR to the RELEASE-NOTE of 5.5.2-SNAPSHOT.

[strongduanmu]

@YaoFly From this PR, we can see that you are familiar with ShardingSphere. If you want to further participate in the development, you can send me your WeChat and I may be able to provide you with more help.

[YaoFly]

@YaoFly From this PR, we can see that you are familiar with ShardingSphere. If you want to further participate in the development, you can send me your WeChat and I may be able to provide you with more help.

wx id : yaoflycn

[strongduanmu]

Looks so great, merged.