Skip to content

Conversation

@xtanuiha
Copy link
Contributor

In our production environment, the mail_message table contains approximately 8.52 million rows, and the res_partner table has about 48,000 rows.

Deleting a single partner took roughly 70 seconds. After adding the database index, the operation was optimized to complete in approximately 3 seconds.

@xtanuiha xtanuiha changed the title [IMP] Optimize database performance for partner deletion [18.0][IMP] Optimize database performance for partner deletion Sep 22, 2025
@OCA-git-bot
Copy link
Contributor

Hi @Kev-Roche,
some modules you are maintaining are being modified, check this out!

@vincent-hatakeyama
Copy link
Contributor

The name of the changed module is missing from the title and the commit message.

@xtanuiha xtanuiha changed the title [18.0][IMP] Optimize database performance for partner deletion [18.0][IMP] impersonate_login:Optimize database performance for partner deletion Sep 25, 2025
@xtanuiha xtanuiha changed the title [18.0][IMP] impersonate_login:Optimize database performance for partner deletion [18.0][IMP] impersonate_login: Optimize database performance for partner deletion Sep 25, 2025
@xtanuiha
Copy link
Contributor Author

The name of the changed module is missing from the title and the commit message.

Thank you for the reminder, the PR and commit message titles have been updated

Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Thanks! Would you be able to port to 19?

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-840-by-StefanRijnhart-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit bf19d77 into OCA:18.0 Dec 28, 2025
8 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a37492d. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants