Skip to content

Conversation

@eantones
Copy link
Member

…ion in _generate_translated_field odoo#18679

It solves issue odoo#18679 (odoo#18679) on Odoo v10

Only works on PostgreSQL 9.5 or above

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

eantones and others added 2 commits January 11, 2020 01:33
@eantones eantones force-pushed the 10.0-odoo-hugely_degraded_performance_in_translation_subquery branch from 9cde369 to 4e19dd0 Compare January 11, 2020 00:38
@FFernandez-PlanetaTIC
Copy link

Hi @eantones,

Will you propose this PR to odoo/OCA repository instead of yours?

If not we can do it.

Regards

@eantones
Copy link
Member Author

Hi @FFernandez-PlanetaTIC ,

you mean to propose to the Odoo official repo, right?

I was going to do that a long time ago but after reading some comments on odoo#18679 like:

odoo#18679 (comment)

odoo#18679 (comment)

odoo#18679 (comment)

I understood that they're never going to merge it, that's why I asked politely in odoo#18679 (comment). But we can try now, at least to have a PR in the official Odoo repo to merge directly on our repos or using tools like git-aggregator. What do you think?

Thanks!

@FFernandez-PlanetaTIC
Copy link

I think in OCA/OCB is more likely to be accepted.

@eantones
Copy link
Member Author

Ok, let's see what OCA/OCB says :-)

OCA#899

Based on PlanetaTIC@3b3c9c6 correction.

- Previous commit breaks import (from ONLY needed in where not exists)
- Avoid duplicates in import (a slightly clearer and cleaner approach)
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.

2 participants