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

8.0 fix qty search perf #569

Open
wants to merge 3 commits into
base: 8.0
Choose a base branch
from

Conversation

mourad-ehm
Copy link

@mourad-ehm mourad-ehm commented Dec 1, 2016

Description of the issue/feature this PR addresses:
Currently search product by quantity is very slow. With this module search time is divided by ~ 200 (tested on 80 000 products). For 80 000 products in databases, search product by quantity take ~ 6.6 mn (400 second). So odoo, compute quantities for each product and make filter in python side. With this PR product are filtered in one SQL request executed in 2 seconds.
The search time remain unchanged with ~ 140 000 products in database.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@StefanRijnhart
Copy link
Member

Impressive stats there! Merging such a big change is quite scary for an understaffed project as OCB, so it might not actually happen until we have reports from people who tested this in production. Have you got a link to a PR on Odoo itself?

@bealdav
Copy link
Member

bealdav commented Jan 30, 2017

@StefanRijnhart This PR is used in production since 3 month at Akretion client

Please have a look it worth. Thanks

cc @jbeficent

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.

None yet

3 participants