-
-
Notifications
You must be signed in to change notification settings - Fork 719
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
[14.0][BACKPORT] [16.0] stock_picking_volume: Speedup install and avoid useless recompute #2148
[14.0][BACKPORT] [16.0] stock_picking_volume: Speedup install and avoid useless recompute #2148
Conversation
Makes computed fields readonly in state 'done' or 'cancel' to avoid triggering recompute when these values are useless
Hi @lmignon, |
Init computed fields from plain sql queries to speedup install on large database
4947b9a
to
3e1537a
Compare
…c' to 'double precision' As no digits parameters are provided on `<stock.move>.volume' and '<stock.picking>.volume' fields, the 'double precision' type is used instead of 'numeric'. This avoids time consuming SQL requests 'ALTER TABLE...' at module install.
9b040d6
to
5fa689a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (Code review only)
This PR has the |
/ocabot merge patch |
This PR looks fantastic, let's merge it! |
It looks like something changed on |
Congratulations, your PR was merged at 93c5fe2. Thanks a lot for contributing to OCA. ❤️ |
Port of #1680 from 16.0 to 14.0.
SQL request in
pre_init_hook
has been adapted to 14.0 (see https://github.com/OCA/OpenUpgrade/blob/16.0/openupgrade_scripts/scripts/stock/16.0.1.1/pre-migration.py#L17-L18 )