Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add DBCOMPATIBILITY 'B' parameter to opengauss database #34062

Merged
merged 38 commits into from
Dec 17, 2024
Merged

Add DBCOMPATIBILITY 'B' parameter to opengauss database #34062

merged 38 commits into from
Dec 17, 2024

Conversation

YaoFly
Copy link
Contributor

@YaoFly YaoFly commented Dec 15, 2024 •
edited
Loading

Fixes #33992.

Changes proposed in this pull request:

  • Add DBCOMPATIBILITY 'B' parameter to opengauss database

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 and others added 30 commits November 26, 2024 11:07
Support GroupConcat sql for aggregating multiple shards(apache#33797)
03ae5ed
Check Style fix(apache#33797)
c0552d1
Check Style fix(apache#33797)
c256c10
spotless fix (apache#33797)
812f813
unit test fix (apache#33797)
05ccc96
@YaoFly
spotless fix (apache#33797)
05eaecb
group_concat distinct compatible (apache#33797)
07209e2
group_concat distinct compatible (apache#33797)
879b107
unit test fix for distinct group_concat (apache#33797)
8feb644
@YaoFly
Merge branch 'apache:master' into master
2201278
@YaoFly
Merge branch 'apache:master' into master
96d9bef
@YaoFly
Merge branch 'apache:master' into master
3efdfb6
e2e test for group_concat (apache#33797)
f0b2f9c
e2e test for group_concat (apache#33797)
4b028ec
Merge remote-tracking branch 'origin/master'
9fb37fa
code format (apache#33797)
91278c4
e2e test (apache#33797)
1f9a97f
e2e test (apache#33797)
c2411b8
e2e test (apache#33797)
ded3745
remove useless code(apache#33797)
c080d71
code optimization (apache#33797)
c96dd07
sql parse unit test (apache#33797)
576cc33
RELEASE-NOTES.md updated(apache#33797)
bd11a26
@YaoFly
Merge branch 'master' into master
a6fe2b2
@YaoFly
Merge branch 'apache:master' into master
370e53d
Code Optimization (apache#33797)
e9ee8e4
@YaoFly
Merge branch 'apache:master' into master
f01105d
Support GroupConcat sql for aggregating multiple shards in opengauss …
fdcdf04 
…and doris database(apache#33797)
doris parse unit test fix (apache#33797)
cf7acbe
spotless fix (apache#33797)
022e56e
@YaoFly YaoFly changed the title Support GroupConcat sql for aggregating multiple shards in opengauss and doris database Add DBCOMPATIBILITY 'B' parameter to opengauss database Dec 15, 2024
@linghengqian linghengqian mentioned this pull request Dec 15, 2024
Open
@YaoFly
Copy link
Contributor Author

YaoFly commented Dec 16, 2024

@strongduanmu PTAL. I have successfully run all e2e cases of opengauss locally.

@strongduanmu
Copy link
Member

Hi @YaoFly, can you add openGauss test case for group_concat? I removed it yesterday

@strongduanmu strongduanmu added this to the 5.5.2 milestone Dec 16, 2024
@YaoFly
Copy link
Contributor Author

YaoFly commented Dec 16, 2024

Hi @YaoFly, can you add openGauss test case for group_concat? I removed it yesterday

Added.

strongduanmu
strongduanmu approved these changes Dec 17, 2024
View reviewed changes
Copy link
Member

@strongduanmu strongduanmu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, mergd.

@strongduanmu strongduanmu merged commit a93e579 into apache:master Dec 17, 2024
135 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@strongduanmu strongduanmu strongduanmu approved these changes

Assignees
No one assigned
Labels
db: openGauss in: test
Projects
None yet
Milestone
5.5.2
Development

Successfully merging this pull request may close these issues.

Support GroupConcat sql for aggregating multiple shards in opengauss and doris database
2 participants
@YaoFly @strongduanmu