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.

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

group_concat() does not consider collatioin resulting errors #27429

Closed
fzhedu opened this issue Aug 20, 2021 · 2 comments · Fixed by #27490
Closed

group_concat() does not consider collatioin resulting errors #27429

fzhedu opened this issue Aug 20, 2021 · 2 comments · Fixed by #27490
Assignees
Labels
severity/critical sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@fzhedu
Copy link
Contributor

fzhedu commented Aug 20, 2021

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

drop table if exists test.t
create table test.t(id int, value varchar(20) charset utf8mb4 collate utf8mb4_general_ci, value1 varchar(20) charset utf8mb4 collate utf8mb4_bin)
alter table test.t set tiflash replica 1
insert into test.t values (1, 'abc', 'abc '),(4, 'Abc', 'abc'),(3,'def', 'def ');

2. What did you expect to see? (Required)

mysql> select /*+ read_from_storage(tiflash[t]) */ upper(group_concat(distinct value order by 1)) from test.t;
+------------------------------------------------+
| upper(group_concat(distinct value order by 1)) |
+------------------------------------------------+
| ABC,DEF                                        |
+------------------------------------------------+
1 row in set (0.01 sec)

3. What did you see instead (Required)

mysql> select /*+ read_from_storage(tikv[t]) */ upper(group_concat(distinct value order by 1)) from test.t;
+------------------------------------------------+
| upper(group_concat(distinct value order by 1)) |
+------------------------------------------------+
| ABC,ABC,DEF                                    |
+------------------------------------------------+
1 row in set (0.00 sec)

4. What is your TiDB version? (Required)

master

@fzhedu fzhedu added type/bug The issue is confirmed as a bug. severity/critical labels Aug 20, 2021
@fzhedu
Copy link
Contributor Author

fzhedu commented Aug 20, 2021

what about other agg functions, like count(distinct x)?

@ti-srebot
Copy link
Contributor

Please edit this comment or add a new comment to complete the following information

Not a bug

  1. Remove the 'type/bug' label
  2. Add notes to indicate why it is not a bug

Duplicate bug

  1. Add the 'type/duplicate' label
  2. Add the link to the original bug

Bug

Note: Make Sure that 'component', and 'severity' labels are added
Example for how to fill out the template: #20100

1. Root Cause Analysis (RCA) (optional)

2. Symptom (optional)

3. All Trigger Conditions (optional)

4. Workaround (optional)

5. Affected versions

6. Fixed versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/critical sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants