Browse Source
Merge pull request #1728 from vauxoo-dev/12.0-ir_sequence_standard_default-ebirbe
Merge pull request #1728 from vauxoo-dev/12.0-ir_sequence_standard_default-ebirbe
[ADD] ir_sequence_standard_default: Use Standard instead No Gap sequence12.0-mig-module_prototyper_last
Moises Lopez - https://www.vauxoo.com/
5 years ago
committed by
GitHub
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
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