<div dir="ltr">Please file a bug report at <a href="https://sourceforge.net/p/adodbapi/bugs/">https://sourceforge.net/p/adodbapi/bugs/</a> .  I will rebuild my test environment which has gradually evaporated over time. Make sure to specify which database engine were you running to find the bug.<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 25, 2016 at 2:53 PM, Max Slimmer <span dir="ltr"><<a href="mailto:maxslimmer@gmail.com" target="_blank">maxslimmer@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Vernon,<div>Yes, that is what I'm saying, I just didn't want to come out shouting "bug". I tried setting up the package test at one point but got stuck somewhere so I'm not sure I'm the person to file the PR (or whatever the process is). But, I'd be happy to file it using whatever the appropriate procedure is (if where we're at isn't enough :)).</div></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Mon, Oct 24, 2016 at 2:57 PM, Vernon D. Cole <span dir="ltr"><<a href="mailto:vernondcole@gmail.com" target="_blank">vernondcole@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Wait a bit...  Did I just hear that there is a difference in operation between starting a connection with autocommit=false and turning autocommit off during a session?  If so, I think that's a bug.  PEP-249 has no hook for explicitly starting a transaction, so adodbapi needs to always have done BeginTrans before any meaningful operation.  Am I missing something?<br>--<br></div>Vernon Cole<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 24, 2016 at 11:10 AM, Tim Roberts <span dir="ltr"><<a href="mailto:timr@probo.com" target="_blank">timr@probo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Max Slimmer wrote:<br>
> The adodbapi documentation and code implies that with autocommit =<br>
> False I should only have to call commit (and possibly rollback) and if<br>
> using the connection as a context manager not even those are required.<br>
> If autocommit = False is passed on the initial connect the __init__ of<br>
> the connector handles BeginTrans and subsequent commit and rollbacks<br>
> also leave me with an open trans.<br>
<br>
Ah, I missed that subtle distinction.  Your modification seems reasonable.<span class="m_-5529973753966758167HOEnZb"><font color="#888888"><br>
<span class="m_-5529973753966758167m_-7950571715061143604HOEnZb"><font color="#888888"><br>
--<br>
</font></span><div class="m_-5529973753966758167m_-7950571715061143604HOEnZb"><div class="m_-5529973753966758167m_-7950571715061143604h5">Tim Roberts, <a href="mailto:timr@probo.com" target="_blank">timr@probo.com</a><br>
Providenza & Boekelheide, Inc.<br>
<br>
______________________________<wbr>_________________<br>
python-win32 mailing list<br>
<a href="mailto:python-win32@python.org" target="_blank">python-win32@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-win32" rel="noreferrer" target="_blank">https://mail.python.org/mailma<wbr>n/listinfo/python-win32</a><br>
</div></div></font></span></blockquote></div><br></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div class="m_-5529973753966758167gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">--Max III</div></div>
</font></span></div>
</blockquote></div><br></div>