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

The memory consumed by UpdateExec.updatedRowKeys and UpdateExec.mergedRowData is not tracked #34096

Closed
ekexium opened this issue Apr 19, 2022 · 1 comment · Fixed by #34097
Closed

The memory consumed by UpdateExec.updatedRowKeys and UpdateExec.mergedRowData is not tracked #34096

ekexium opened this issue Apr 19, 2022 · 1 comment · Fixed by #34097
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@ekexium
Copy link
Contributor

ekexium commented Apr 19, 2022 •
edited
Loading

Enhancement

updatedRowKeys and mergedRowData in UpdateExec can use a large amount of memory but is not tracked in the memory tracker.

update `T_CUSTOMER` set purse_num = purse_num + 1 where vip_level = '0'
image

update T_CUSTOMER n, T_CUSTOMER m set n.purse_num=1, m.purse_num=2 where n.vip_level = '0'
image
@ekexium ekexium added the type/enhancement The issue or PR belongs to an enhancement. label Apr 19, 2022
@ekexium ekexium mentioned this issue Apr 19, 2022
Merged
12 tasks
@ekexium ekexium changed the title The memory consumed by UpdateExec.updatedRowKeys is not tracked The memory consumed by UpdateExec.updatedRowKeys and UpdateExec.mergedRowData is not tracked Apr 21, 2022
@XuHuaiyu
Copy link
Contributor

refer #33877

ti-chi-bot pushed a commit that referenced this issue Jun 22, 2022
*: track the memory usage in Insert/Update/Delete executors (#34097)
95e13af 
close #34096
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

*: track the memory usage in Insert/Update/Delete executors ekexium/tidb
2 participants
@XuHuaiyu @ekexium