<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 09/18/2016 01:01 PM, Thomas De
Schampheleire wrote:<br>
</div>
<blockquote
cite="mid:CAAXf6LVhZirP_x5LJ9iuvnW6+16X4LVZKOd=eEoVZvq2T4xAkQ@mail.gmail.com"
type="cite">
<p dir="ltr">On Sep 18, 2016 12:28, "Alessandro Molina" <<a
moz-do-not-send="true"
href="mailto:alessandro.molina@gmail.com">alessandro.molina@gmail.com</a>>
wrote:<br>
><br>
> If I remember correctly kallithea explicitly handles
commits, so probably it has no need for the transaction manager.<br>
> Try to add base_config['tm.enabled'] = False in app_cfg.py
and see it everything still works as expected and the error
disappeared.</p>
<p dir="ltr">Thanks, I will try that.<br>
Do you have more details about this transaction manager? Which
package normally supplies this, how does it work?<br>
</p>
</blockquote>
<br>
(I guess it is as mentioned on
<a class="moz-txt-link-freetext" href="https://turbogears.readthedocs.io/en/development/reference/upgrading.html#transaction-manager-is-now-an-application-wrapper">https://turbogears.readthedocs.io/en/development/reference/upgrading.html#transaction-manager-is-now-an-application-wrapper</a>
)<br>
<br>
<blockquote
cite="mid:CAAXf6LVhZirP_x5LJ9iuvnW6+16X4LVZKOd=eEoVZvq2T4xAkQ@mail.gmail.com"
type="cite">
<p dir="ltr">
Would it replace existing calls to Session.commit(), or is it
not as simple as that?<br>
And finally, in your opinion, is it better to use Kallithea's
current approach, or consider using transaction manager?<br>
</p>
</blockquote>
<br>
I think that is related to an outstanding issue from
<a class="moz-txt-link-freetext" href="https://bitbucket.org/conservancy/kallithea/pull-requests/249/activate-vcs-opertations-tests-v4/diff">https://bitbucket.org/conservancy/kallithea/pull-requests/249/activate-vcs-opertations-tests-v4/diff</a>
, as discussed on
<a class="moz-txt-link-freetext" href="https://bitbucket.org/domruf/kallithea/commits/06f66664f210ffed83a4b52a07528000572cc9a1?at=default#comment-3676506">https://bitbucket.org/domruf/kallithea/commits/06f66664f210ffed83a4b52a07528000572cc9a1?at=default#comment-3676506</a>
. We need better control over our database transactions.<br>
<br>
It would probably be a good idea to move towards having HTTP
requests and database transactions tied closely together. Kallithea
is however not there yet, so I guess this 'tm' should be disabled
for now. (Looking at AppConfig, we should perhaps use 'minimal'? I
don't know if it is better to rely on things "being taken care of"
(but where we still have to know what is being taken care of and how
it works), or if it is better to be more explicit.)<br>
<br>
/Mads<br>
</body>
</html>