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

[FIX] base: optimize property fields #28769

Closed
wants to merge 1 commit into
base: 11.0
from

Conversation

Projects
None yet
4 participants
@rco-odoo
Member

rco-odoo commented Nov 16, 2018

Implement the getter method in plain SQL. This makes reading company-dependent
fields 33% faster (on 20000 partners, reading 'property_product_pricelist'
takes 3700ms instead of 5500ms.)

opw-1906628

@robodoo robodoo added the seen 🙂 label Nov 16, 2018

@C3POdoo C3POdoo added the RD label Nov 16, 2018

@robodoo robodoo added the CI 🤖 label Nov 16, 2018

@rco-odoo rco-odoo force-pushed the odoo-dev:11.0-faster-property-rco branch from 4f91d94 to 511210b Nov 19, 2018

@robodoo robodoo removed the CI 🤖 label Nov 19, 2018

@rco-odoo rco-odoo force-pushed the odoo-dev:11.0-faster-property-rco branch from 511210b to 7f68513 Nov 19, 2018

@robodoo robodoo added the CI 🤖 label Nov 19, 2018

@rco-odoo rco-odoo force-pushed the odoo-dev:11.0-faster-property-rco branch from 7f68513 to e3125e1 Nov 19, 2018

@robodoo robodoo removed the CI 🤖 label Nov 19, 2018

[FIX] base: optimize property fields
Implement the getter method in plain SQL.  This makes reading company-dependent
fields 33% faster (on 20000 partners, reading 'property_product_pricelist'
takes 3700ms instead of 5500ms.)

@rco-odoo rco-odoo force-pushed the odoo-dev:11.0-faster-property-rco branch from e3125e1 to ff9d98e Nov 19, 2018

@robodoo robodoo added the CI 🤖 label Nov 19, 2018

@rco-odoo

This comment has been minimized.

Member

rco-odoo commented Nov 20, 2018

robodoo r+

@robodoo robodoo added the r+ 👌 label Nov 20, 2018

robodoo pushed a commit that referenced this pull request Nov 20, 2018

[FIX] base: optimize property fields
Implement the getter method in plain SQL.  This makes reading company-dependent
fields 33% faster (on 20000 partners, reading 'property_product_pricelist'
takes 3700ms instead of 5500ms.)

closes #28769

@rim-odoo rim-odoo self-requested a review Nov 20, 2018

@robodoo robodoo added merged 🎉 and removed merging 👷 labels Nov 20, 2018

@robodoo

This comment has been minimized.

Contributor

robodoo commented Nov 20, 2018

Merged, thanks!

@robodoo robodoo closed this Nov 20, 2018

@rco-odoo rco-odoo deleted the odoo-dev:11.0-faster-property-rco branch Nov 20, 2018

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