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

Forward Port of #36492 to saas-12.3 #36660

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Sep 10, 2019

[FIX] orm: in modified() force target search ordering by 'id'

Some model can have really complex _order, which can induce unnecessary slow
queries; here as we don't need the right business order we force ordering by
'id' instead

OPW-2007478

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

Forward-Port-Of: #36492

Some model can have really complex _order, which can induce unnecessary slow
queries; here as we don't need the right business order we force ordering by
'id' instead

OPW-2007478

X-original-commit: a1fe897
@robodoo
Copy link
Contributor

robodoo commented Sep 10, 2019

Ping @xavieralt, @rco-odoo

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Sep 10, 2019
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Sep 10, 2019
@rco-odoo
Copy link
Member

@robodoo r+

robodoo pushed a commit that referenced this pull request Sep 11, 2019
Some model can have really complex _order, which can induce unnecessary slow
queries; here as we don't need the right business order we force ordering by
'id' instead

OPW-2007478

closes #36660

X-original-commit: a1fe897
Signed-off-by: Raphael Collet (rco) <rco@openerp.com>
@robodoo
Copy link
Contributor

robodoo commented Sep 11, 2019

Merged at 5032fc1, thanks!

@robodoo robodoo closed this Sep 11, 2019
@rco-odoo rco-odoo deleted the saas-12.3-11.0-2007478-faster-modified-xal-56OYSGBA-forwardport branch September 11, 2019 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants