changeset:   106164:74a68d86569c
branch:      2.7
parent:      106160:a87bf2d630ed
user:        Benjamin Peterson <benjamin@python.org>
date:        Mon Jan 16 00:07:27 2017 -0800
files:       Lib/sqlite3/test/regression.py Misc/NEWS Modules/_sqlite/connection.c
description:
revert dd13098a5dc2 (#29006, #10513)


diff -r a87bf2d630ed -r 74a68d86569c Lib/sqlite3/test/regression.py
--- a/Lib/sqlite3/test/regression.py	Mon Jan 16 00:05:12 2017 -0800
+++ b/Lib/sqlite3/test/regression.py	Mon Jan 16 00:07:27 2017 -0800
@@ -328,6 +328,36 @@
         self.assertRaises(ValueError, cur.execute, " \0select 2")
         self.assertRaises(ValueError, cur.execute, "select 2\0")
 
+    def CheckCommitCursorReset(self):
+        """
+        Connection.commit() did reset cursors, which made sqlite3
+        to return rows multiple times when fetched from cursors
+        after commit. See issues 10513 and 23129 for details.
+        """
+        con = sqlite.connect(":memory:")
+        con.executescript("""
+        create table t(c);
+        create table t2(c);
+        insert into t values(0);
+        insert into t values(1);
+        insert into t values(2);
+        """)
+
+        self.assertEqual(con.isolation_level, "")
+
+        counter = 0
+        for i, row in enumerate(con.execute("select c from t")):
+            con.execute("insert into t2(c) values (?)", (i,))
+            con.commit()
+            if counter == 0:
+                self.assertEqual(row[0], 0)
+            elif counter == 1:
+                self.assertEqual(row[0], 1)
+            elif counter == 2:
+                self.assertEqual(row[0], 2)
+            counter += 1
+        self.assertEqual(counter, 3, "should have returned exactly three rows")
+
 
 def suite():
     regression_suite = unittest.makeSuite(RegressionTests, "Check")
diff -r a87bf2d630ed -r 74a68d86569c Misc/NEWS
--- a/Misc/NEWS	Mon Jan 16 00:05:12 2017 -0800
+++ b/Misc/NEWS	Mon Jan 16 00:07:27 2017 -0800
@@ -29,9 +29,6 @@
 - Issue #29082: Fixed loading libraries in ctypes by unicode names on Windows.
   Original patch by Chi Hsuan Yen.
 
-- Issue #29006: Revert change from issue #10513 for making sqlite more liable to
-  emit "database table is locked" errors.
-
 - Issue #29188: Support glibc 2.24 on Linux: don't use getentropy() function
   but read from /dev/urandom to get random bytes, for example in os.urandom().
   On Linux, getentropy() is implemented which getrandom() is blocking mode,
@@ -257,6 +254,9 @@
 
 - Issue #19884: Avoid spurious output on OS X with Gnu Readline.
 
+- Issue #10513: Fix a regression in Connection.commit().  Statements should
+  not be reset after a commit.
+
 - Issue #2466: posixpath.ismount now correctly recognizes mount points which
   the user does not have permission to access.
 
diff -r a87bf2d630ed -r 74a68d86569c Modules/_sqlite/connection.c
--- a/Modules/_sqlite/connection.c	Mon Jan 16 00:05:12 2017 -0800
+++ b/Modules/_sqlite/connection.c	Mon Jan 16 00:07:27 2017 -0800
@@ -467,7 +467,6 @@
     }
 
     if (self->inTransaction) {
-        pysqlite_do_all_statements(self, ACTION_RESET, 0);
 
         Py_BEGIN_ALLOW_THREADS
         rc = sqlite3_prepare(self->db, "COMMIT", -1, &statement, &tail);