Skip to content

(8.4) PS-9647: MySQL Perf Improvements (deque)#5640

Merged
inikep merged 1 commit into
percona:8.4from
kamil-holubicki:PS-9647-deque-8.4
Jul 28, 2025
Merged

(8.4) PS-9647: MySQL Perf Improvements (deque)#5640
inikep merged 1 commit into
percona:8.4from
kamil-holubicki:PS-9647-deque-8.4

Conversation

@kamil-holubicki

Copy link
Copy Markdown
Contributor

https://perconadev.atlassian.net/browse/PS-9647

This patch uses list instead of deque for memory efficiency reasons, primarily to optimize operations like bulk insert

Cherry pick of 7509e14

@percona-ysorokin percona-ysorokin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (1/1)

Comment thread sql/sql_optimizer.cc
emb_sj_nest->nested_join->sj_inner_exprs;
for (size_t i = 0; i < items.size(); i++) {
const Item *sel_item = items[i];
size_t i = 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ readability-identifier-length ⚠️
variable name i is too short, expected at least 2 characters

https://perconadev.atlassian.net/browse/PS-9647

This patch uses list instead of deque for memory efficiency reasons,
primarily to optimize operations like bulk insert

@percona-ysorokin percona-ysorokin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@inikep
inikep merged commit aa1a059 into percona:8.4 Jul 28, 2025
21 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants