[issue30593] sqlite3 executescript does not respect isolation_level?

[Erlend E. Aasland]

Change by Erlend E. Aasland :


--
pull_requests: +24837
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/26220

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Erlend E. Aasland]

Erlend E. Aasland  added the comment:

Mentioning this behaviour in the documentation should be sufficient.

--
assignee:  -> docs@python
components: +Documentation -Library (Lib)
nosy: +berker.peksag, docs@python, erlendaasland
versions: +Python 3.10, Python 3.11, Python 3.9 -Python 2.7, Python 3.4, Python 
3.5, Python 3.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Aviv Palivoda]

Changes by Aviv Palivoda :


--
nosy: +palaviv

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Noah Levitt]

Changes by Noah Levitt :


Added file: http://bugs.python.org/file46943/issue30593-test.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Noah Levitt]

Noah Levitt added the comment:

And here's a fix.

Unfortunately I think the change could break people's scripts. If they have 
isolation_level set (not None) and use executescript(), they will have to issue 
an explicit call to connection.commit().

executescript() could do the commit itself explicitly after running the sql, I 
suppose. Then the behavior is much like the old behavior, but at least you get 
the performance boost of running inside a transaction. (Substantial if you pass 
10k insert statements to executescript(), or something like that.)

Or, we could not change the code, and instead change the documentation. The 
docs could suggest wrapping the sql script with "begin;" and "commit;".

--
keywords: +patch
Added file: http://bugs.python.org/file46942/issue30593-fix.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Noah Levitt]

Noah Levitt added the comment:

Here's a test case

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Jim Jewett]

Jim Jewett added the comment:

Do you have a test case that could be used to verify a patch?
(Or even a full patch?)

--
nosy: +Jim.Jewett
stage:  -> needs patch
type:  -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30593] sqlite3 executescript does not respect isolation_level?

[Noah Levitt]

New submission from Noah Levitt:

As far as I can tell, sqlite3 executescript() does not respect isolation_level. 
Is that true? If so, I think it's worth mentioning in the doc. Or maybe it 
should respect isolation_level, not sure there's any particular reason not to.

--
components: Library (Lib)
messages: 295376
nosy: Noah Levitt
priority: normal
severity: normal
status: open
title: sqlite3 executescript does not respect isolation_level?
versions: Python 2.7, Python 3.4, Python 3.5, Python 3.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com