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

bug: delete from Multiple-Table, Failed to delete the data #663

Closed
2 of 3 tasks
davidshiz opened this issue Oct 9, 2022 · 0 comments · Fixed by #746
Closed
2 of 3 tasks

bug: delete from Multiple-Table, Failed to delete the data #663

davidshiz opened this issue Oct 9, 2022 · 0 comments · Fixed by #746
Assignees
@konghaiya
Labels
A-bug Something isn't working
Milestone
StoneDB_5.7_v1.0.1

Comments

@davidshiz
Copy link
Collaborator

Have you read the Contributing Guidelines on issues?

Please confirm if bug report does NOT exists already ?

  • I confirm there is no existing issue for this

Describe the problem

Failed to delete the data!
image

Expected behavior

image

How To Reproduce

create table t11 (a int NOT NULL, b int, primary key (a))ENGINE=TIANMU;
create table t12 (a int NOT NULL, b int, primary key (a))ENGINE=TIANMU;
insert into t11 values (0, 10),(1, 11),(2, 12);
insert into t12 values (33, 10),(0, 11),(2, 12);

select * from t11,t12;
delete t11.*,t12.* from t11,t12 where t11.a = t12.a;

Environment

ubuntu@ubuntu:~$ cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.4 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.4 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

ubuntu@ubuntu:~$ /stonedb57/install/bin/mysqld --version
/stonedb57/install/bin/mysqld Ver 5.7.36_v1.0.1_beta_2-StoneDB for Linux on x86_64 (build-)
build information as follow:
Repository address: https://github.com/stoneatom/stonedb.git:stonedb-5.7-dev
Branch name: stonedb-5.7-dev
Last commit ID: 208f79d
Last commit time: Date: Sat Sep 17 16:19:42 2022 +0800
Build time: Date: Fri Sep 23 14:14:27 CST 2022

Are you interested in submitting a PR to solve the problem?

  • Yes, I will!
@davidshiz davidshiz added the A-bug Something isn't working label Oct 9, 2022
@davidshiz davidshiz added this to the stonedb_5.7_v1.0.1 milestone Oct 9, 2022
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
Fix delete from Multiple-Table, Failed to delete the data (stoneatom#663
52bdaa3 
)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
85ac8d4 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
@konghaiya konghaiya mentioned this issue Oct 18, 2022
Merged
15 tasks
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
5d3fbb9 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
65a88e4 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
72e8635 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
1faf41c 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
1891460 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 18, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
86ed23f 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 19, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
47a4dbd 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 19, 2022
@konghaiya
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (s…
862cf27 
…toneatom#663)(stoneatom#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
konghaiya added a commit to konghaiya/stonedb that referenced this issue Oct 20, 2022
@konghaiya
fix(tianmu): Fix delete from Multiple-Table, Failed to delete the data (
e4b1071 
stoneatom#663)
@mergify mergify bot closed this as completed in #746 Oct 20, 2022
mergify bot pushed a commit that referenced this issue Oct 20, 2022
@konghaiya @mergify
fix(tianmu)Fix delete from Multiple-Table, Failed to delete the data (#…
ea99be9 
…663)(#678)

Cause:
At present, the primary key of the tianmu engine does not fully support delete and update statements.
Temporary solution:
Modify the execution plan of the SQL layer so that the delete and update statements do not follow the primary key logic
mergify bot pushed a commit that referenced this issue Oct 20, 2022
@konghaiya @mergify
fix(tianmu): Fix delete from Multiple-Table, Failed to delete the data (
24a27db 
#663)
@hustjieke hustjieke mentioned this issue Mar 19, 2023
Closed
@hustjieke hustjieke mentioned this issue May 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@konghaiya konghaiya

Labels
A-bug Something isn't working
Projects
None yet
Milestone
StoneDB_5.7_v1.0.1
Development

Successfully merging a pull request may close this issue.

fix(tianmu): Fix delete from Multiple-Table, Failed to delete the data (#663) konghaiya/stonedb
2 participants
@davidshiz @konghaiya