Skip to content

[typo](docs)Delete unsupported sql statements in GROUP_CONCAT() - #21455

Merged
BiteTheDDDDt merged 2 commits into
apache:masterfrom
echo-hhj:doris_doc
Jul 4, 2023
Merged

[typo](docs)Delete unsupported sql statements in GROUP_CONCAT()#21455
BiteTheDDDDt merged 2 commits into
apache:masterfrom
echo-hhj:doris_doc

Conversation

@echo-hhj

@echo-hhjecho-hhj commented Jul 3, 2023

Copy link
Copy Markdown
Contributor

Proposed changes

I deleted the case in GROUP_CONCAT () and modified the corresponding English and Chinese documents

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

I deleted the case in GROUP_CONCAT () and modified the corresponding English and Chinese documents
I deleted the case in GROUP_CONCAT () and modified the corresponding English and Chinese documents
@github-actionsgithub-actionsBot added the kind/docs Categorizes issue or PR as related to documentation. label Jul 3, 2023
@echo-hhjecho-hhj closed this Jul 3, 2023
@echo-hhjecho-hhj reopened this Jul 3, 2023
@echo-hhjecho-hhj closed this Jul 3, 2023
@echo-hhjecho-hhj reopened this Jul 3, 2023
@BiteTheDDDDt

Copy link
Copy Markdown
Contributor

why remove it? It is supported, are you having any issues?

@echo-hhj

Copy link
Copy Markdown
ContributorAuthor

why remove it? It is supported, are you having any issues?

Sorry for the reply error, test this sql execution failed in 126rc version

@echo-hhj

Copy link
Copy Markdown
ContributorAuthor

why remove it? It is supported, are you having any issues?

截屏2023-07-03 16 19 59

@BiteTheDDDDtBiteTheDDDDt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, this support removed on #17237

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Jul 3, 2023
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@zy-kkk

Copy link
Copy Markdown
Member

run buildall

@BiteTheDDDDt
BiteTheDDDDt merged commit 890e55b into apache:masterJul 4, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.kind/docsCategorizes issue or PR as related to documentation.reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@echo-hhj@BiteTheDDDDt@zy-kkk