Browse Source
[ADD] ir_sequence_standard_default: Use Standard instead No Gap sequence
[ADD] ir_sequence_standard_default: Use Standard instead No Gap sequence
Set the implementation to "Standard" in all your current Sequences (ir.sequence) and all new sequences are created as "Standard" by default instead of "No Gap" implementation. **What's the problem Sequences with "No Gap" Implementation?** "No Gap" is the default value of sequences in Odoo. However, this kind of sequences cause more locks and can turn a database slow. Taking as example an invoice, if you assign an invoice number to one record, but it sill not finish the process, this process must end in order to another invoice could assign a new number and there was no gaps between the invoice numbers. It seems to be good at first sight. But the problem start when there is and chained process. Imagine that there is one user that executes a process that produces 100 invoices and these at the same time produces 100 journal entries that also use a consecutive (no gap) sequence. And also those invoices are sent to sign with and external institution (that could take 2 seconds in giving a response because of internet latency or server load), and maybe they made another calculations that makes them to take 5 seconds more for each invoice, and all this is chained to one single transaction. This means that for 8.5 minutes anybody else could confirm invoices, neither journal entries of the involved journals. Now, think there is 20 users that have to execute a similar process. The problem turns exponential. If another user comes to make an operation with the same jornal it will thrown a concurrency failure. You can mitigate it if you segment each transaction and don't chain them. It means, making commit for each invoice or process. It reduces the probability that there is a concurrency error or a lock wait. However, it still not solve it completely. **Why to use Sequences with "Standard" Implementation?** If you use the standard sequence of PosgreSQL, it doesn't lock because at the moment the request is done, the next sequence number it is changed in an isolated transaction, and it have not to wait the other transaction to end. However, if the transaction produces a rollback, this sequence isn't reverted, it means, it's lost. It may be not not serious because when you cancel or remove records that number is lost too. **What this module does?** To eliminate completely that concurrency/slowness problem, this module changes all the sequences (ir.sequence) implementation from "No Gap" to "Standard" with the awareness that it will skip numbers. In the majority of database models and many users projects there is no problem with that jump occurs. Also, all newly created sequences will be by default in "Standard" implementation.12.0-mig-module_prototyper_last
Erick Birbe
5 years ago
15 changed files with 856 additions and 0 deletions
-
137ir_sequence_standard_default/README.rst
-
6ir_sequence_standard_default/__init__.py
-
23ir_sequence_standard_default/__manifest__.py
-
4ir_sequence_standard_default/models/__init__.py
-
16ir_sequence_standard_default/models/res_config_settings.py
-
3ir_sequence_standard_default/readme/CONTRIBUTORS.rst
-
52ir_sequence_standard_default/readme/DESCRIPTION.rst
-
BINir_sequence_standard_default/static/description/icon.png
-
455ir_sequence_standard_default/static/description/index.html
-
5ir_sequence_standard_default/tests/__init__.py
-
46ir_sequence_standard_default/tests/test_sequence_standard_default.py
-
31ir_sequence_standard_default/views/res_config_settings_views.xml
-
4ir_sequence_standard_default/wizard/__init__.py
-
42ir_sequence_standard_default/wizard/sequence_standard_default.py
-
32ir_sequence_standard_default/wizard/sequence_standard_default_views.xml
@ -0,0 +1,137 @@ |
|||||
|
============================== |
||||
|
IrSequence Standard by Default |
||||
|
============================== |
||||
|
|
||||
|
.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! |
||||
|
!! This file is generated by oca-gen-addon-readme !! |
||||
|
!! changes will be overwritten. !! |
||||
|
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! |
||||
|
|
||||
|
.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png |
||||
|
:target: https://odoo-community.org/page/development-status |
||||
|
:alt: Beta |
||||
|
.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png |
||||
|
:target: http://www.gnu.org/licenses/agpl-3.0-standalone.html |
||||
|
:alt: License: AGPL-3 |
||||
|
.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fserver--tools-lightgray.png?logo=github |
||||
|
:target: https://github.com/OCA/server-tools/tree/12.0/ir_sequence_standard_default |
||||
|
:alt: OCA/server-tools |
||||
|
.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png |
||||
|
:target: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-ir_sequence_standard_default |
||||
|
:alt: Translate me on Weblate |
||||
|
.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png |
||||
|
:target: https://runbot.odoo-community.org/runbot/149/12.0 |
||||
|
:alt: Try me on Runbot |
||||
|
|
||||
|
|badge1| |badge2| |badge3| |badge4| |badge5| |
||||
|
|
||||
|
Set the implementation to "Standard" in all your current Sequences |
||||
|
(ir.sequence) and all new sequences are created as "Standard" by default |
||||
|
instead of "No Gap" implementation. |
||||
|
|
||||
|
What's the problem with "No Gap" Sequence Implementations |
||||
|
========================================================= |
||||
|
|
||||
|
"No Gap" is the default value of sequences in Odoo. However, this kind of |
||||
|
sequences cause more locks and can turn a database slow. |
||||
|
|
||||
|
Taking as example an invoice, if you assign an invoice number to one record, |
||||
|
but it sill not finish the process, this process must end in order to another |
||||
|
invoice could assign a new number and there was no gaps between the invoice |
||||
|
numbers. It seems to be good at first sight. But the problem starts when there |
||||
|
is a chained process. |
||||
|
|
||||
|
Imagine that there is one user that executes a process that produces 100 |
||||
|
invoices and these at the same time produces 100 journal entries that also use |
||||
|
a consecutive (no gap) sequence. And also those invoices are sent to sign with |
||||
|
and external institution (that could take 2 seconds in giving a response |
||||
|
because of internet latency or server load), and maybe they made another |
||||
|
calculations that makes them to take 5 seconds more for each invoice, and all |
||||
|
this is chained to one single transaction. This means that for 8.5 minutes |
||||
|
anybody else could confirm invoices, neither journal entries of the involved |
||||
|
journals. |
||||
|
|
||||
|
Now, think there is 20 users that have to execute a similar process. The |
||||
|
problem turns exponential. If another user comes to make an operation with the |
||||
|
same jornal it will thrown a concurrency failure. |
||||
|
|
||||
|
You can mitigate it if you segment each transaction and don't chain them. It |
||||
|
means, making commit for each invoice or process. It reduces the |
||||
|
probability that there is a concurrency error or a lock wait. However, it still |
||||
|
not solve it completely. |
||||
|
|
||||
|
Why to use Sequences with "Standard" Implementation |
||||
|
=================================================== |
||||
|
|
||||
|
If you use the standard sequence of PosgreSQL, it doesn't lock because at the |
||||
|
moment the request is done, the next sequence number it is changed in an |
||||
|
isolated transaction, and it have not to wait the other transaction to end. |
||||
|
However, if the transaction produces a rollback, this sequence isn't reverted, |
||||
|
it means, it's lost. It may be not not serious because when you cancel or |
||||
|
remove records that number is lost too. |
||||
|
|
||||
|
What this module does |
||||
|
===================== |
||||
|
|
||||
|
To eliminate completely that concurrency/slowness problem, this module changes |
||||
|
all the sequences (ir.sequence) implementation from "No Gap" to "Standard" with |
||||
|
the awareness that it will skip numbers. In the majority of database models |
||||
|
and many users projects there is no problem with that jump occurs. |
||||
|
|
||||
|
**Table of contents** |
||||
|
|
||||
|
.. contents:: |
||||
|
:local: |
||||
|
|
||||
|
Bug Tracker |
||||
|
=========== |
||||
|
|
||||
|
Bugs are tracked on `GitHub Issues <https://github.com/OCA/server-tools/issues>`_. |
||||
|
In case of trouble, please check there if your issue has already been reported. |
||||
|
If you spotted it first, help us smashing it by providing a detailed and welcomed |
||||
|
`feedback <https://github.com/OCA/server-tools/issues/new?body=module:%20ir_sequence_standard_default%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_. |
||||
|
|
||||
|
Do not contact contributors directly about support or help with technical issues. |
||||
|
|
||||
|
Credits |
||||
|
======= |
||||
|
|
||||
|
Authors |
||||
|
~~~~~~~ |
||||
|
|
||||
|
* Vauxoo |
||||
|
|
||||
|
Contributors |
||||
|
~~~~~~~~~~~~ |
||||
|
|
||||
|
- Moises López <moylop260@vauxoo.com> |
||||
|
- Erick Birbe <erick@vauxoo.com> |
||||
|
|
||||
|
|
||||
|
Maintainers |
||||
|
~~~~~~~~~~~ |
||||
|
|
||||
|
This module is maintained by the OCA. |
||||
|
|
||||
|
.. image:: https://odoo-community.org/logo.png |
||||
|
:alt: Odoo Community Association |
||||
|
:target: https://odoo-community.org |
||||
|
|
||||
|
OCA, or the Odoo Community Association, is a nonprofit organization whose |
||||
|
mission is to support the collaborative development of Odoo features and |
||||
|
promote its widespread use. |
||||
|
|
||||
|
.. |maintainer-moylop260| image:: https://github.com/moylop260.png?size=40px |
||||
|
:target: https://github.com/moylop260 |
||||
|
:alt: moylop260 |
||||
|
.. |maintainer-ebirbe| image:: https://github.com/ebirbe.png?size=40px |
||||
|
:target: https://github.com/ebirbe |
||||
|
:alt: ebirbe |
||||
|
|
||||
|
Current `maintainers <https://odoo-community.org/page/maintainer-role>`__: |
||||
|
|
||||
|
|maintainer-moylop260| |maintainer-ebirbe| |
||||
|
|
||||
|
This module is part of the `OCA/server-tools <https://github.com/OCA/server-tools/tree/12.0/ir_sequence_standard_default>`_ project on GitHub. |
||||
|
|
||||
|
You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. |
@ -0,0 +1,6 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
from . import models |
||||
|
from . import wizard |
||||
|
from . import tests |
@ -0,0 +1,23 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
|
||||
|
{ |
||||
|
'name': 'IrSequence Standard by Default', |
||||
|
'summary': 'Use Standard implementation of ir.sequence instead of NoGap', |
||||
|
'version': '12.0.1.0.0', |
||||
|
'author': 'Vauxoo, Odoo Community Association (OCA)', |
||||
|
'website': 'https://github.com/OCA/server-tools', |
||||
|
'maintainers': ['moylop260', 'ebirbe'], |
||||
|
'license': 'AGPL-3', |
||||
|
'category': 'Tools', |
||||
|
'depends': [ |
||||
|
'base_setup', |
||||
|
], |
||||
|
'data': [ |
||||
|
'views/res_config_settings_views.xml', |
||||
|
'wizard/sequence_standard_default_views.xml', |
||||
|
], |
||||
|
'installable': True, |
||||
|
'application': False, |
||||
|
} |
@ -0,0 +1,4 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
from . import res_config_settings |
@ -0,0 +1,16 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
from odoo import api, models |
||||
|
|
||||
|
|
||||
|
class ResConfigSettings(models.TransientModel): |
||||
|
|
||||
|
_inherit = "res.config.settings" |
||||
|
|
||||
|
@api.multi |
||||
|
def action_change_all_sequences(self): |
||||
|
self.ensure_one() |
||||
|
action = self.env.ref( |
||||
|
'ir_sequence_standard_default.action_sequence_standard_default') |
||||
|
return action.read()[0] |
@ -0,0 +1,3 @@ |
|||||
|
- Moises López <moylop260@vauxoo.com> |
||||
|
- Erick Birbe <erick@vauxoo.com> |
||||
|
|
@ -0,0 +1,52 @@ |
|||||
|
Set the implementation to "Standard" in all your current Sequences |
||||
|
(ir.sequence) and all new sequences are created as "Standard" by default |
||||
|
instead of "No Gap" implementation. |
||||
|
|
||||
|
What's the problem with "No Gap" Sequence Implementations |
||||
|
========================================================= |
||||
|
|
||||
|
"No Gap" is the default value of sequences in Odoo. However, this kind of |
||||
|
sequences cause more locks and can turn a database slow. |
||||
|
|
||||
|
Taking as example an invoice, if you assign an invoice number to one record, |
||||
|
but it sill not finish the process, this process must end in order to another |
||||
|
invoice could assign a new number and there was no gaps between the invoice |
||||
|
numbers. It seems to be good at first sight. But the problem starts when there |
||||
|
is a chained process. |
||||
|
|
||||
|
Imagine that there is one user that executes a process that produces 100 |
||||
|
invoices and these at the same time produces 100 journal entries that also use |
||||
|
a consecutive (no gap) sequence. And also those invoices are sent to sign with |
||||
|
and external institution (that could take 2 seconds in giving a response |
||||
|
because of internet latency or server load), and maybe they made another |
||||
|
calculations that makes them to take 5 seconds more for each invoice, and all |
||||
|
this is chained to one single transaction. This means that for 8.5 minutes |
||||
|
anybody else could confirm invoices, neither journal entries of the involved |
||||
|
journals. |
||||
|
|
||||
|
Now, think there is 20 users that have to execute a similar process. The |
||||
|
problem turns exponential. If another user comes to make an operation with the |
||||
|
same jornal it will thrown a concurrency failure. |
||||
|
|
||||
|
You can mitigate it if you segment each transaction and don't chain them. It |
||||
|
means, making commit for each invoice or process. It reduces the |
||||
|
probability that there is a concurrency error or a lock wait. However, it still |
||||
|
not solve it completely. |
||||
|
|
||||
|
Why to use Sequences with "Standard" Implementation |
||||
|
=================================================== |
||||
|
|
||||
|
If you use the standard sequence of PosgreSQL, it doesn't lock because at the |
||||
|
moment the request is done, the next sequence number it is changed in an |
||||
|
isolated transaction, and it have not to wait the other transaction to end. |
||||
|
However, if the transaction produces a rollback, this sequence isn't reverted, |
||||
|
it means, it's lost. It may be not not serious because when you cancel or |
||||
|
remove records that number is lost too. |
||||
|
|
||||
|
What this module does |
||||
|
===================== |
||||
|
|
||||
|
To eliminate completely that concurrency/slowness problem, this module changes |
||||
|
all the sequences (ir.sequence) implementation from "No Gap" to "Standard" with |
||||
|
the awareness that it will skip numbers. In the majority of database models |
||||
|
and many users projects there is no problem with that jump occurs. |
After Width: 128 | Height: 128 | Size: 9.2 KiB |
@ -0,0 +1,455 @@ |
|||||
|
<?xml version="1.0" encoding="utf-8" ?> |
||||
|
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> |
||||
|
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"> |
||||
|
<head> |
||||
|
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> |
||||
|
<meta name="generator" content="Docutils 0.15.1: http://docutils.sourceforge.net/" /> |
||||
|
<title>IrSequence Standard by Default</title> |
||||
|
<style type="text/css"> |
||||
|
|
||||
|
/* |
||||
|
:Author: David Goodger (goodger@python.org) |
||||
|
:Id: $Id: html4css1.css 7952 2016-07-26 18:15:59Z milde $ |
||||
|
:Copyright: This stylesheet has been placed in the public domain. |
||||
|
|
||||
|
Default cascading style sheet for the HTML output of Docutils. |
||||
|
|
||||
|
See http://docutils.sf.net/docs/howto/html-stylesheets.html for how to |
||||
|
customize this style sheet. |
||||
|
*/ |
||||
|
|
||||
|
/* used to remove borders from tables and images */ |
||||
|
.borderless, table.borderless td, table.borderless th { |
||||
|
border: 0 } |
||||
|
|
||||
|
table.borderless td, table.borderless th { |
||||
|
/* Override padding for "table.docutils td" with "! important". |
||||
|
The right padding separates the table cells. */ |
||||
|
padding: 0 0.5em 0 0 ! important } |
||||
|
|
||||
|
.first { |
||||
|
/* Override more specific margin styles with "! important". */ |
||||
|
margin-top: 0 ! important } |
||||
|
|
||||
|
.last, .with-subtitle { |
||||
|
margin-bottom: 0 ! important } |
||||
|
|
||||
|
.hidden { |
||||
|
display: none } |
||||
|
|
||||
|
.subscript { |
||||
|
vertical-align: sub; |
||||
|
font-size: smaller } |
||||
|
|
||||
|
.superscript { |
||||
|
vertical-align: super; |
||||
|
font-size: smaller } |
||||
|
|
||||
|
a.toc-backref { |
||||
|
text-decoration: none ; |
||||
|
color: black } |
||||
|
|
||||
|
blockquote.epigraph { |
||||
|
margin: 2em 5em ; } |
||||
|
|
||||
|
dl.docutils dd { |
||||
|
margin-bottom: 0.5em } |
||||
|
|
||||
|
object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] { |
||||
|
overflow: hidden; |
||||
|
} |
||||
|
|
||||
|
/* Uncomment (and remove this text!) to get bold-faced definition list terms |
||||
|
dl.docutils dt { |
||||
|
font-weight: bold } |
||||
|
*/ |
||||
|
|
||||
|
div.abstract { |
||||
|
margin: 2em 5em } |
||||
|
|
||||
|
div.abstract p.topic-title { |
||||
|
font-weight: bold ; |
||||
|
text-align: center } |
||||
|
|
||||
|
div.admonition, div.attention, div.caution, div.danger, div.error, |
||||
|
div.hint, div.important, div.note, div.tip, div.warning { |
||||
|
margin: 2em ; |
||||
|
border: medium outset ; |
||||
|
padding: 1em } |
||||
|
|
||||
|
div.admonition p.admonition-title, div.hint p.admonition-title, |
||||
|
div.important p.admonition-title, div.note p.admonition-title, |
||||
|
div.tip p.admonition-title { |
||||
|
font-weight: bold ; |
||||
|
font-family: sans-serif } |
||||
|
|
||||
|
div.attention p.admonition-title, div.caution p.admonition-title, |
||||
|
div.danger p.admonition-title, div.error p.admonition-title, |
||||
|
div.warning p.admonition-title, .code .error { |
||||
|
color: red ; |
||||
|
font-weight: bold ; |
||||
|
font-family: sans-serif } |
||||
|
|
||||
|
/* Uncomment (and remove this text!) to get reduced vertical space in |
||||
|
compound paragraphs. |
||||
|
div.compound .compound-first, div.compound .compound-middle { |
||||
|
margin-bottom: 0.5em } |
||||
|
|
||||
|
div.compound .compound-last, div.compound .compound-middle { |
||||
|
margin-top: 0.5em } |
||||
|
*/ |
||||
|
|
||||
|
div.dedication { |
||||
|
margin: 2em 5em ; |
||||
|
text-align: center ; |
||||
|
font-style: italic } |
||||
|
|
||||
|
div.dedication p.topic-title { |
||||
|
font-weight: bold ; |
||||
|
font-style: normal } |
||||
|
|
||||
|
div.figure { |
||||
|
margin-left: 2em ; |
||||
|
margin-right: 2em } |
||||
|
|
||||
|
div.footer, div.header { |
||||
|
clear: both; |
||||
|
font-size: smaller } |
||||
|
|
||||
|
div.line-block { |
||||
|
display: block ; |
||||
|
margin-top: 1em ; |
||||
|
margin-bottom: 1em } |
||||
|
|
||||
|
div.line-block div.line-block { |
||||
|
margin-top: 0 ; |
||||
|
margin-bottom: 0 ; |
||||
|
margin-left: 1.5em } |
||||
|
|
||||
|
div.sidebar { |
||||
|
margin: 0 0 0.5em 1em ; |
||||
|
border: medium outset ; |
||||
|
padding: 1em ; |
||||
|
background-color: #ffffee ; |
||||
|
width: 40% ; |
||||
|
float: right ; |
||||
|
clear: right } |
||||
|
|
||||
|
div.sidebar p.rubric { |
||||
|
font-family: sans-serif ; |
||||
|
font-size: medium } |
||||
|
|
||||
|
div.system-messages { |
||||
|
margin: 5em } |
||||
|
|
||||
|
div.system-messages h1 { |
||||
|
color: red } |
||||
|
|
||||
|
div.system-message { |
||||
|
border: medium outset ; |
||||
|
padding: 1em } |
||||
|
|
||||
|
div.system-message p.system-message-title { |
||||
|
color: red ; |
||||
|
font-weight: bold } |
||||
|
|
||||
|
div.topic { |
||||
|
margin: 2em } |
||||
|
|
||||
|
h1.section-subtitle, h2.section-subtitle, h3.section-subtitle, |
||||
|
h4.section-subtitle, h5.section-subtitle, h6.section-subtitle { |
||||
|
margin-top: 0.4em } |
||||
|
|
||||
|
h1.title { |
||||
|
text-align: center } |
||||
|
|
||||
|
h2.subtitle { |
||||
|
text-align: center } |
||||
|
|
||||
|
hr.docutils { |
||||
|
width: 75% } |
||||
|
|
||||
|
img.align-left, .figure.align-left, object.align-left, table.align-left { |
||||
|
clear: left ; |
||||
|
float: left ; |
||||
|
margin-right: 1em } |
||||
|
|
||||
|
img.align-right, .figure.align-right, object.align-right, table.align-right { |
||||
|
clear: right ; |
||||
|
float: right ; |
||||
|
margin-left: 1em } |
||||
|
|
||||
|
img.align-center, .figure.align-center, object.align-center { |
||||
|
display: block; |
||||
|
margin-left: auto; |
||||
|
margin-right: auto; |
||||
|
} |
||||
|
|
||||
|
table.align-center { |
||||
|
margin-left: auto; |
||||
|
margin-right: auto; |
||||
|
} |
||||
|
|
||||
|
.align-left { |
||||
|
text-align: left } |
||||
|
|
||||
|
.align-center { |
||||
|
clear: both ; |
||||
|
text-align: center } |
||||
|
|
||||
|
.align-right { |
||||
|
text-align: right } |
||||
|
|
||||
|
/* reset inner alignment in figures */ |
||||
|
div.align-right { |
||||
|
text-align: inherit } |
||||
|
|
||||
|
/* div.align-center * { */ |
||||
|
/* text-align: left } */ |
||||
|
|
||||
|
.align-top { |
||||
|
vertical-align: top } |
||||
|
|
||||
|
.align-middle { |
||||
|
vertical-align: middle } |
||||
|
|
||||
|
.align-bottom { |
||||
|
vertical-align: bottom } |
||||
|
|
||||
|
ol.simple, ul.simple { |
||||
|
margin-bottom: 1em } |
||||
|
|
||||
|
ol.arabic { |
||||
|
list-style: decimal } |
||||
|
|
||||
|
ol.loweralpha { |
||||
|
list-style: lower-alpha } |
||||
|
|
||||
|
ol.upperalpha { |
||||
|
list-style: upper-alpha } |
||||
|
|
||||
|
ol.lowerroman { |
||||
|
list-style: lower-roman } |
||||
|
|
||||
|
ol.upperroman { |
||||
|
list-style: upper-roman } |
||||
|
|
||||
|
p.attribution { |
||||
|
text-align: right ; |
||||
|
margin-left: 50% } |
||||
|
|
||||
|
p.caption { |
||||
|
font-style: italic } |
||||
|
|
||||
|
p.credits { |
||||
|
font-style: italic ; |
||||
|
font-size: smaller } |
||||
|
|
||||
|
p.label { |
||||
|
white-space: nowrap } |
||||
|
|
||||
|
p.rubric { |
||||
|
font-weight: bold ; |
||||
|
font-size: larger ; |
||||
|
color: maroon ; |
||||
|
text-align: center } |
||||
|
|
||||
|
p.sidebar-title { |
||||
|
font-family: sans-serif ; |
||||
|
font-weight: bold ; |
||||
|
font-size: larger } |
||||
|
|
||||
|
p.sidebar-subtitle { |
||||
|
font-family: sans-serif ; |
||||
|
font-weight: bold } |
||||
|
|
||||
|
p.topic-title { |
||||
|
font-weight: bold } |
||||
|
|
||||
|
pre.address { |
||||
|
margin-bottom: 0 ; |
||||
|
margin-top: 0 ; |
||||
|
font: inherit } |
||||
|
|
||||
|
pre.literal-block, pre.doctest-block, pre.math, pre.code { |
||||
|
margin-left: 2em ; |
||||
|
margin-right: 2em } |
||||
|
|
||||
|
pre.code .ln { color: grey; } /* line numbers */ |
||||
|
pre.code, code { background-color: #eeeeee } |
||||
|
pre.code .comment, code .comment { color: #5C6576 } |
||||
|
pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold } |
||||
|
pre.code .literal.string, code .literal.string { color: #0C5404 } |
||||
|
pre.code .name.builtin, code .name.builtin { color: #352B84 } |
||||
|
pre.code .deleted, code .deleted { background-color: #DEB0A1} |
||||
|
pre.code .inserted, code .inserted { background-color: #A3D289} |
||||
|
|
||||
|
span.classifier { |
||||
|
font-family: sans-serif ; |
||||
|
font-style: oblique } |
||||
|
|
||||
|
span.classifier-delimiter { |
||||
|
font-family: sans-serif ; |
||||
|
font-weight: bold } |
||||
|
|
||||
|
span.interpreted { |
||||
|
font-family: sans-serif } |
||||
|
|
||||
|
span.option { |
||||
|
white-space: nowrap } |
||||
|
|
||||
|
span.pre { |
||||
|
white-space: pre } |
||||
|
|
||||
|
span.problematic { |
||||
|
color: red } |
||||
|
|
||||
|
span.section-subtitle { |
||||
|
/* font-size relative to parent (h1..h6 element) */ |
||||
|
font-size: 80% } |
||||
|
|
||||
|
table.citation { |
||||
|
border-left: solid 1px gray; |
||||
|
margin-left: 1px } |
||||
|
|
||||
|
table.docinfo { |
||||
|
margin: 2em 4em } |
||||
|
|
||||
|
table.docutils { |
||||
|
margin-top: 0.5em ; |
||||
|
margin-bottom: 0.5em } |
||||
|
|
||||
|
table.footnote { |
||||
|
border-left: solid 1px black; |
||||
|
margin-left: 1px } |
||||
|
|
||||
|
table.docutils td, table.docutils th, |
||||
|
table.docinfo td, table.docinfo th { |
||||
|
padding-left: 0.5em ; |
||||
|
padding-right: 0.5em ; |
||||
|
vertical-align: top } |
||||
|
|
||||
|
table.docutils th.field-name, table.docinfo th.docinfo-name { |
||||
|
font-weight: bold ; |
||||
|
text-align: left ; |
||||
|
white-space: nowrap ; |
||||
|
padding-left: 0 } |
||||
|
|
||||
|
/* "booktabs" style (no vertical lines) */ |
||||
|
table.docutils.booktabs { |
||||
|
border: 0px; |
||||
|
border-top: 2px solid; |
||||
|
border-bottom: 2px solid; |
||||
|
border-collapse: collapse; |
||||
|
} |
||||
|
table.docutils.booktabs * { |
||||
|
border: 0px; |
||||
|
} |
||||
|
table.docutils.booktabs th { |
||||
|
border-bottom: thin solid; |
||||
|
text-align: left; |
||||
|
} |
||||
|
|
||||
|
h1 tt.docutils, h2 tt.docutils, h3 tt.docutils, |
||||
|
h4 tt.docutils, h5 tt.docutils, h6 tt.docutils { |
||||
|
font-size: 100% } |
||||
|
|
||||
|
ul.auto-toc { |
||||
|
list-style-type: none } |
||||
|
|
||||
|
</style> |
||||
|
</head> |
||||
|
<body> |
||||
|
<div class="document" id="irsequence-standard-by-default"> |
||||
|
<h1 class="title">IrSequence Standard by Default</h1> |
||||
|
|
||||
|
<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! |
||||
|
!! This file is generated by oca-gen-addon-readme !! |
||||
|
!! changes will be overwritten. !! |
||||
|
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! --> |
||||
|
<p><a class="reference external" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external" href="https://github.com/OCA/server-tools/tree/12.0/ir_sequence_standard_default"><img alt="OCA/server-tools" src="https://img.shields.io/badge/github-OCA%2Fserver--tools-lightgray.png?logo=github" /></a> <a class="reference external" href="https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-ir_sequence_standard_default"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external" href="https://runbot.odoo-community.org/runbot/149/12.0"><img alt="Try me on Runbot" src="https://img.shields.io/badge/runbot-Try%20me-875A7B.png" /></a></p> |
||||
|
<p>Set the implementation to “Standard” in all your current Sequences |
||||
|
(ir.sequence) and all new sequences are created as “Standard” by default |
||||
|
instead of “No Gap” implementation.</p> |
||||
|
<div class="section" id="what-s-the-problem-with-no-gap-sequence-implementations"> |
||||
|
<h1>What’s the problem with “No Gap” Sequence Implementations</h1> |
||||
|
<p>“No Gap” is the default value of sequences in Odoo. However, this kind of |
||||
|
sequences cause more locks and can turn a database slow.</p> |
||||
|
<p>Taking as example an invoice, if you assign an invoice number to one record, |
||||
|
but it sill not finish the process, this process must end in order to another |
||||
|
invoice could assign a new number and there was no gaps between the invoice |
||||
|
numbers. It seems to be good at first sight. But the problem starts when there |
||||
|
is a chained process.</p> |
||||
|
<p>Imagine that there is one user that executes a process that produces 100 |
||||
|
invoices and these at the same time produces 100 journal entries that also use |
||||
|
a consecutive (no gap) sequence. And also those invoices are sent to sign with |
||||
|
and external institution (that could take 2 seconds in giving a response |
||||
|
because of internet latency or server load), and maybe they made another |
||||
|
calculations that makes them to take 5 seconds more for each invoice, and all |
||||
|
this is chained to one single transaction. This means that for 8.5 minutes |
||||
|
anybody else could confirm invoices, neither journal entries of the involved |
||||
|
journals.</p> |
||||
|
<p>Now, think there is 20 users that have to execute a similar process. The |
||||
|
problem turns exponential. If another user comes to make an operation with the |
||||
|
same jornal it will thrown a concurrency failure.</p> |
||||
|
<p>You can mitigate it if you segment each transaction and don’t chain them. It |
||||
|
means, making commit for each invoice or process. It reduces the |
||||
|
probability that there is a concurrency error or a lock wait. However, it still |
||||
|
not solve it completely.</p> |
||||
|
</div> |
||||
|
<div class="section" id="why-to-use-sequences-with-standard-implementation"> |
||||
|
<h1>Why to use Sequences with “Standard” Implementation</h1> |
||||
|
<p>If you use the standard sequence of PosgreSQL, it doesn’t lock because at the |
||||
|
moment the request is done, the next sequence number it is changed in an |
||||
|
isolated transaction, and it have not to wait the other transaction to end. |
||||
|
However, if the transaction produces a rollback, this sequence isn’t reverted, |
||||
|
it means, it’s lost. It may be not not serious because when you cancel or |
||||
|
remove records that number is lost too.</p> |
||||
|
</div> |
||||
|
<div class="section" id="what-this-module-does"> |
||||
|
<h1>What this module does</h1> |
||||
|
<p>To eliminate completely that concurrency/slowness problem, this module changes |
||||
|
all the sequences (ir.sequence) implementation from “No Gap” to “Standard” with |
||||
|
the awareness that it will skip numbers. In the majority of database models |
||||
|
and many users projects there is no problem with that jump occurs.</p> |
||||
|
<p><strong>Table of contents</strong></p> |
||||
|
</div> |
||||
|
<div class="section" id="bug-tracker"> |
||||
|
<h1>Bug Tracker</h1> |
||||
|
<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/server-tools/issues">GitHub Issues</a>. |
||||
|
In case of trouble, please check there if your issue has already been reported. |
||||
|
If you spotted it first, help us smashing it by providing a detailed and welcomed |
||||
|
<a class="reference external" href="https://github.com/OCA/server-tools/issues/new?body=module:%20ir_sequence_standard_default%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p> |
||||
|
<p>Do not contact contributors directly about support or help with technical issues.</p> |
||||
|
</div> |
||||
|
<div class="section" id="credits"> |
||||
|
<h1>Credits</h1> |
||||
|
<div class="section" id="authors"> |
||||
|
<h2>Authors</h2> |
||||
|
<ul class="simple"> |
||||
|
<li>Vauxoo</li> |
||||
|
</ul> |
||||
|
</div> |
||||
|
<div class="section" id="contributors"> |
||||
|
<h2>Contributors</h2> |
||||
|
<ul class="simple"> |
||||
|
<li>Moises López <<a class="reference external" href="mailto:moylop260@vauxoo.com">moylop260@vauxoo.com</a>></li> |
||||
|
<li>Erick Birbe <<a class="reference external" href="mailto:erick@vauxoo.com">erick@vauxoo.com</a>></li> |
||||
|
</ul> |
||||
|
</div> |
||||
|
<div class="section" id="maintainers"> |
||||
|
<h2>Maintainers</h2> |
||||
|
<p>This module is maintained by the OCA.</p> |
||||
|
<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a> |
||||
|
<p>OCA, or the Odoo Community Association, is a nonprofit organization whose |
||||
|
mission is to support the collaborative development of Odoo features and |
||||
|
promote its widespread use.</p> |
||||
|
<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainers</a>:</p> |
||||
|
<p><a class="reference external" href="https://github.com/moylop260"><img alt="moylop260" src="https://github.com/moylop260.png?size=40px" /></a> <a class="reference external" href="https://github.com/ebirbe"><img alt="ebirbe" src="https://github.com/ebirbe.png?size=40px" /></a></p> |
||||
|
<p>This module is part of the <a class="reference external" href="https://github.com/OCA/server-tools/tree/12.0/ir_sequence_standard_default">OCA/server-tools</a> project on GitHub.</p> |
||||
|
<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p> |
||||
|
</div> |
||||
|
</div> |
||||
|
</div> |
||||
|
</body> |
||||
|
</html> |
@ -0,0 +1,5 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
from . import test_sequence_standard_default |
||||
|
|
@ -0,0 +1,46 @@ |
|||||
|
# Copyright 2016 Vauxoo |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
from odoo.tests.common import TransactionCase |
||||
|
|
||||
|
|
||||
|
class TestSequenceStandardDefault(TransactionCase): |
||||
|
|
||||
|
def setUp(self): |
||||
|
|
||||
|
super(TestSequenceStandardDefault, self).setUp() |
||||
|
self.wizard = self.env['sequence.standard.default'].create({}) |
||||
|
self.seq_nogap = self.env['ir.sequence'].create({ |
||||
|
'name': 'NoGap Sequence', |
||||
|
'implementation': 'no_gap', |
||||
|
'use_date_range': False, |
||||
|
'number_next_actual': 100, |
||||
|
}) |
||||
|
self.seq_nogap_range = self.env['ir.sequence'].create({ |
||||
|
'name': 'NoGap Sequence with Date Range', |
||||
|
'implementation': 'no_gap', |
||||
|
'use_date_range': True, |
||||
|
}) |
||||
|
self.date_range = self.env['ir.sequence.date_range'].create({ |
||||
|
'sequence_id': self.seq_nogap_range.id, |
||||
|
'date_from': '2019-01-01', |
||||
|
'date_to': '2019-12-31', |
||||
|
'number_next_actual': 200, |
||||
|
}) |
||||
|
self.seq_standard = self.env['ir.sequence'].create({ |
||||
|
'name': 'Standard Sequence', |
||||
|
'implementation': 'standard', |
||||
|
'number_next_actual': 300, |
||||
|
}) |
||||
|
|
||||
|
def test01_nogap_to_std(self): |
||||
|
self.wizard.execute() |
||||
|
|
||||
|
self.assertEqual(self.seq_nogap.implementation, 'standard') |
||||
|
self.assertEqual(self.seq_nogap.number_next_actual, 100) |
||||
|
|
||||
|
self.assertEqual(self.seq_nogap_range.implementation, 'standard') |
||||
|
self.assertEqual(self.date_range.number_next_actual, 200) |
||||
|
|
||||
|
self.assertEqual(self.seq_standard.implementation, 'standard') |
||||
|
self.assertEqual(self.seq_standard.number_next_actual, 300) |
@ -0,0 +1,31 @@ |
|||||
|
<?xml version="1.0" encoding="utf-8"?> |
||||
|
<odoo> |
||||
|
|
||||
|
<record id="res_config_settings_view_form" model="ir.ui.view"> |
||||
|
<field name="name">res.config.settings.view.form.inherit.sequence</field> |
||||
|
<field name="model">res.config.settings</field> |
||||
|
<field name="inherit_id" ref="base_setup.res_config_settings_view_form" /> |
||||
|
<field name="arch" type="xml"> |
||||
|
<xpath expr="//div[@name='integration']" position="after"> |
||||
|
<div id="sequences_general_settings" groups="base.group_no_one"> |
||||
|
<h2>Sequences</h2> |
||||
|
<div class="row mt16 o_settings_container"> |
||||
|
<div class="col-12 col-lg-6 o_setting_box"> |
||||
|
<div class="o_setting_left_pane"/> |
||||
|
<div class="o_setting_right_pane"> |
||||
|
<span class="o_form_label">Force Standard</span> |
||||
|
<div class="text-muted"> |
||||
|
Change all your current No Gap sequences to Standard in order to improve performance. |
||||
|
</div> |
||||
|
<div class="mt16"> |
||||
|
<button name="action_change_all_sequences" string="Change all sequences to Standard implementation" type="object" class="oe_link" icon="fa-arrow-right"/> |
||||
|
</div> |
||||
|
</div> |
||||
|
</div> |
||||
|
</div> |
||||
|
</div> |
||||
|
</xpath> |
||||
|
</field> |
||||
|
</record> |
||||
|
|
||||
|
</odoo> |
@ -0,0 +1,4 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
from . import sequence_standard_default |
@ -0,0 +1,42 @@ |
|||||
|
# 2019 Vauxoo (<http://www.vauxoo.com/>) |
||||
|
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). |
||||
|
|
||||
|
import logging |
||||
|
from psycopg2.extensions import AsIs |
||||
|
|
||||
|
from odoo import api, models |
||||
|
|
||||
|
_logger = logging.getLogger(__name__) |
||||
|
|
||||
|
|
||||
|
class SequenceStandardDefault(models.TransientModel): |
||||
|
|
||||
|
_name = "sequence.standard.default" |
||||
|
_description = "Wizard to set all sequences to Standard implementation" |
||||
|
|
||||
|
@api.model |
||||
|
def change_all_sequences(self): |
||||
|
sequences = self.env['ir.sequence'].sudo().with_context( |
||||
|
active_test=False).search([('implementation', '=', 'no_gap')]) |
||||
|
_logger.info("Changing sequences to Standard: %s", sequences.ids) |
||||
|
for item in sequences: |
||||
|
seq_name = "ir_sequence_%03d" % (item.id) |
||||
|
self._cr.execute("DROP SEQUENCE IF EXISTS %s", [AsIs(seq_name), ]) |
||||
|
if item.use_date_range: |
||||
|
range_ids = {} |
||||
|
for line in item.date_range_ids: |
||||
|
range_ids[line.id] = line.number_next_actual |
||||
|
seq_name = "ir_sequence_%03d_%03d" % (item.id, line.id) |
||||
|
self._cr.execute( |
||||
|
"DROP SEQUENCE IF EXISTS %s", [AsIs(seq_name), ]) |
||||
|
item.write({'implementation': 'standard'}) |
||||
|
for line in item.date_range_ids: |
||||
|
line.write({'number_next_actual': range_ids[line.id]}) |
||||
|
continue |
||||
|
item.write({'implementation': 'standard'}) |
||||
|
|
||||
|
@api.multi |
||||
|
def execute(self): |
||||
|
self.ensure_one() |
||||
|
self.change_all_sequences() |
||||
|
return self.env.ref('base.ir_sequence_form').read()[0] |
@ -0,0 +1,32 @@ |
|||||
|
<?xml version="1.0" encoding="utf-8"?> |
||||
|
<odoo> |
||||
|
|
||||
|
<record id="view_sequence_standard_default" model="ir.ui.view"> |
||||
|
<field name="name">Set All Sequences to Standard</field> |
||||
|
<field name="model">sequence.standard.default</field> |
||||
|
<field name="arch" type="xml"> |
||||
|
<form> |
||||
|
<h2>Warning:</h2> |
||||
|
<p>All sequences in the system will be changed to the Standard |
||||
|
implementation of PostgreSQL. It will improve performance, |
||||
|
but sequence numbers may have gaps between each other.</p> |
||||
|
<p>What do you want to do?</p> |
||||
|
<footer> |
||||
|
<button name="execute" string="Execute" type="object" class="btn-primary"/> |
||||
|
<button special="cancel" string="Cancel" class="btn-secondary"/> |
||||
|
</footer> |
||||
|
</form> |
||||
|
</field> |
||||
|
</record> |
||||
|
|
||||
|
<record id="action_sequence_standard_default" model="ir.actions.act_window"> |
||||
|
<field name="name">Set All Sequences to Standard</field> |
||||
|
<field name="type">ir.actions.act_window</field> |
||||
|
<field name="res_model">sequence.standard.default</field> |
||||
|
<field name="view_type">form</field> |
||||
|
<field name="view_mode">form</field> |
||||
|
<field name="target">new</field> |
||||
|
</record> |
||||
|
|
||||
|
</odoo> |
||||
|
|
Write
Preview
Loading…
Cancel
Save
Reference in new issue