diff options
| author | Xavier Morel <xmo@odoo.com> | 2019-06-04 10:55:12 +0000 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2019-06-19 13:43:53 +0000 |
| commit | c5c955db7acd94ae4682a1e5040320806ca22056 (patch) | |
| tree | 7a83940c311e6808a3a32de8cb259d46b0fe2db9 /odoo/netsvc.py | |
| parent | 455714c7bf77fb9b650f086aa937cafae979f2e8 (diff) | |
[FIX] core: work around ir_logging deadlock
DROP CONSTRAINT (even with IF EXISTS is specified) acquires an ACCESS
EXCLUSIVE lock on the table, preventing e.g. inserts in an other
transaction, so ir_logging would systematically deadlock if configured
to the same database and a warning would be triggered during install
or update (if that ran ir.logging's init).
1. hand-roll the "IF EXISTS" bit, to avoid taking an ACCESS EXCLUSIVE
lock on the table if the problematic constraint does not exist and
thus doesn't need to be dropped (which by now should be the vast
majority of cases).
Replacing DROP CONSTRAINT with DISABLE TRIGGER does not fix the
issue as *that* acquires SHARE ROW EXCLUSIVE. While that's less
constraitning than ACCESS EXCLUSIVE, it still conflicts with an
insert's ROW_EXCLUSIVE.
2. add a timeout to the logging INSERT anyway, the deadlock is still
an issue if we're updating a database which does have the
problematic constraint, and we want to preclude the possible
eventual introduction of new deadlocks in the future.
closes odoo/odoo#34243
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'odoo/netsvc.py')
| -rw-r--r-- | odoo/netsvc.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/odoo/netsvc.py b/odoo/netsvc.py index 84e9702479b..dac9b433e8c 100644 --- a/odoo/netsvc.py +++ b/odoo/netsvc.py @@ -54,7 +54,7 @@ def LocalService(name): path_prefix = os.path.realpath(os.path.dirname(os.path.dirname(__file__))) class PostgreSQLHandler(logging.Handler): - """ PostgreSQL Loggin Handler will store logs in the database, by default + """ PostgreSQL Logging Handler will store logs in the database, by default the current database, can be set using --log-db=DBNAME """ def emit(self, record): @@ -64,7 +64,8 @@ class PostgreSQLHandler(logging.Handler): if not dbname: return with tools.ignore(Exception), tools.mute_logger('odoo.sql_db'), sql_db.db_connect(dbname, allow_uri=True).cursor() as cr: - cr.autocommit(True) + # preclude risks of deadlocks + cr.execute("SET LOCAL statement_timeout = 1000") msg = tools.ustr(record.msg) if record.args: msg = msg % record.args |