launchpad-reviewers team mailing list archive

[İlker Emre Koç <mp+491360@xxxxxxxxxxxxxxxxxx>]

İlker Emre Koç has proposed merging ~ilkeremrekoc/launchpad:librarian-gc-cursor-fix into launchpad:master.

Commit message:
delete cursor name for multiple query executions

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ilkeremrekoc/launchpad/+git/launchpad/+merge/491360
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilkeremrekoc/launchpad:librarian-gc-cursor-fix into launchpad:master.

diff --git a/lib/lp/services/librarianserver/librariangc.py b/lib/lp/services/librarianserver/librariangc.py
index 7f40457..5c04c55 100644
--- a/lib/lp/services/librarianserver/librariangc.py
+++ b/lib/lp/services/librarianserver/librariangc.py
@@ -911,7 +911,7 @@ def delete_unwanted_swift_files(con):
     finally:
         cur.close()
 
-    cur = con.cursor(name="librariangc_swift_lfcs")
+    cur = con.cursor()  # nameless cursor for multiple executions
 
     # Calculate all stored LibraryFileContent ids that we want to keep.
     # Results are ordered so we don't have to suck them all in at once.