Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(8)

Issue 411271003: tryton-env: Catch get_connection errors in Cache listener (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 months, 3 weeks ago by nicoe
Modified:
4 months ago
Reviewers:
rietveld-bot, ced, reviewbot
Visibility:
Public.

Description

tryton-env: Catch get_connection errors in Cache listener issue11563

Patch Set 1 #

Total comments: 2

Patch Set 2 : Move database handling in try / finally #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -10 lines) Patch
M trytond/trytond/cache.py View 1 2 chunks +16 lines, -10 lines 0 comments Download

Messages

Total messages: 11
nicoe
7 months, 3 weeks ago (2022-06-10 14:25:55 UTC) #1
reviewbot
checks OK URL: https://codereview.tryton.org/411271003
7 months, 3 weeks ago (2022-06-10 14:35:03 UTC) #2
ced
https://codereview.tryton.org/411271003/diff/421321003/trytond/trytond/cache.py File trytond/trytond/cache.py (right): https://codereview.tryton.org/411271003/diff/421321003/trytond/trytond/cache.py#newcode353 trytond/trytond/cache.py:353: raise NotImplementedError I'm wondering if we should not put ...
7 months, 3 weeks ago (2022-06-10 15:31:23 UTC) #3
nicoe
6 months, 4 weeks ago (2022-07-07 14:58:26 UTC) #4
nicoe
https://codereview.tryton.org/411271003/diff/421321003/trytond/trytond/cache.py File trytond/trytond/cache.py (right): https://codereview.tryton.org/411271003/diff/421321003/trytond/trytond/cache.py#newcode353 trytond/trytond/cache.py:353: raise NotImplementedError On 2022/06/10 15:31:23, ced wrote: > I'm ...
6 months, 4 weeks ago (2022-07-07 14:58:31 UTC) #5
reviewbot
checks OK URL: https://codereview.tryton.org/411271003
6 months, 4 weeks ago (2022-07-07 15:16:05 UTC) #6
ced
LGTM
6 months, 2 weeks ago (2022-07-18 20:39:44 UTC) #7
ced
On 2022/07/18 20:39:44, ced wrote: > LGTM ping
5 months, 3 weeks ago (2022-08-08 20:30:40 UTC) #8
rietveld-bot_tryton.org
New changeset 7123014d260c by Nicolas Évrard in branch 'default': Catch get_connection errors in Cache listener ...
4 months, 2 weeks ago (2022-09-17 17:00:59 UTC) #9
rietveld-bot_tryton.org
New changeset 59d883262042 by Nicolas Évrard in branch 'default': Catch get_connection errors in Cache listener ...
4 months, 2 weeks ago (2022-09-17 17:01:03 UTC) #10
rietveld-bot_tryton.org
4 months ago (2022-09-29 14:58:20 UTC) #11
New changeset a87d1206c5cc by Nicolas Évrard in branch '6.4':
Catch get_connection errors in Cache listener
https://hg.tryton.org/trytond/rev/a87d1206c5cc


New changeset ae0476e33d82 by Nicolas Évrard in branch '6.2':
Catch get_connection errors in Cache listener
https://hg.tryton.org/trytond/rev/ae0476e33d82


New changeset ffac357d4312 by Nicolas Évrard in branch '6.0':
Catch get_connection errors in Cache listener
https://hg.tryton.org/trytond/rev/ffac357d4312
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld d9ca037-tainted