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

Support drop index sql bind and add test case. #34165

Merged
merged 42 commits into from
Dec 26, 2024
Merged

Support drop index sql bind and add test case. #34165

merged 42 commits into from
Dec 26, 2024

Conversation

YaoFly
Copy link
Contributor

@YaoFly YaoFly commented Dec 26, 2024

Fixes #34029.

Changes proposed in this pull request:

  • Support drop index sql bind and add test case.

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
strongduanmu and others added 12 commits December 10, 2024 19:19
@strongduanmu strongduanmu added this to the 5.5.2 milestone Dec 26, 2024
strongduanmu
strongduanmu approved these changes Dec 26, 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.

LGTM, merged.

@strongduanmu strongduanmu merged commit 1073a04 into apache:master Dec 26, 2024
147 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
in: SQL bind type: enhancement
Projects
None yet
Milestone
5.5.2
Development

Successfully merging this pull request may close these issues.

Add sql bind logic for TableAvailable to simplify TablesContext logic
2 participants
@YaoFly @strongduanmu