Skip to content

MDEV-39226: Add multi-table update, delete feature - #5529

Draft
bsrikanth-mariadb wants to merge 1 commit into
mainfrom
13.2-MDEV-39226-direct-multi_table-update-delete
Draft

MDEV-39226: Add multi-table update, delete feature#5529
bsrikanth-mariadb wants to merge 1 commit into
mainfrom
13.2-MDEV-39226-direct-multi_table-update-delete

Conversation

@bsrikanth-mariadb

Copy link
Copy Markdown
Contributor

No description provided.

@bsrikanth-mariadb
bsrikanth-mariadb marked this pull request as draft August 11, 2026 10:54
@bsrikanth-mariadb
bsrikanth-mariadb force-pushed the 13.2-MDEV-39226-direct-multi_table-update-delete branch 5 times, most recently from f2ce815 to 44765e4 Compare August 14, 2026 08:16
Comment thread sql/sql_explain.cc Outdated
const char *pushed_select_text= "PUSHED SELECT";
const char *pushed_update_text= "PUSHED DOWN UPDATE";
const char *pushed_delete_text= "PUSHED DOWN DELETE";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please follow the pattern: if we use PUSHED SELECT, let's add PUSHED UPDATE , not PUSHED DOWN UPDATE.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, true. But, felt "PUSHED DOWN XXXX" was sounding better.

Anyways, will keep them consistent.

@bsrikanth-mariadb
bsrikanth-mariadb force-pushed the 13.2-MDEV-39226-direct-multi_table-update-delete branch from 44765e4 to 466e400 Compare August 14, 2026 10:08
@bsrikanth-mariadb
bsrikanth-mariadb force-pushed the 13.2-MDEV-39226-direct-multi_table-update-delete branch from 466e400 to b931524 Compare August 14, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants