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

improve lock meta table for DML #20763

Merged
merged 4 commits into from
Dec 16, 2024

Conversation

ouyuanning
Copy link
Contributor

@ouyuanning ouyuanning commented Dec 13, 2024

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #20622 #20721

What this PR does / why we need it:

improve lock meta table for DML
1、复用上meta锁的vector
2、去掉上锁时候每次都获取relation的逻辑,提前拿table id
3、当要上多个表的meta锁的时候,合并到一个vector

@ouyuanning
Copy link
Contributor Author

本地单机tpcc 10仓4线程,1分钟来看。
约有2%的提升。(本分支和main 各测2次,取平均值)

@mergify mergify bot added the kind/refactor Code refactor label Dec 13, 2024
@mergify mergify bot merged commit fa32fcb into matrixorigin:main Dec 16, 2024
18 checks passed
ouyuanning added a commit to ouyuanning/matrixone that referenced this pull request Dec 16, 2024
improve lock meta table for DML
1、复用上meta锁的vector
2、去掉上锁时候每次都获取relation的逻辑,提前拿table id
3、当要上多个表的meta锁的时候,合并到一个vector

Approved by: @zhangxu19830126, @badboynt1
mergify bot pushed a commit that referenced this pull request Dec 16, 2024
improve lock meta table for DML (#20763)

Approved by: @badboynt1, @sukki37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/refactor Code refactor size/M Denotes a PR that changes [100,499] lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants