[Collab-qa-commits] r1537 - udd/udd

[Lucas Nussbaum]

Author: lucas
Date: 2009-07-24 12:04:21 +0000 (Fri, 24 Jul 2009)
New Revision: 1537

Modified:
   udd/udd/upload_history_gatherer.py
Log:
use several executemany()

Modified: udd/udd/upload_history_gatherer.py
===================================================================
--- udd/udd/upload_history_gatherer.py	2009-07-24 11:00:34 UTC (rev 1536)
+++ udd/udd/upload_history_gatherer.py	2009-07-24 12:04:21 UTC (rev 1537)
@@ -51,20 +51,19 @@
       %(Fingerprint)s)"
     query_archs = "EXECUTE uh_arch_insert(%(id)s, %(arch)s)"
     query_closes = "EXECUTE uh_close_insert(%(id)s, %(closes)s)"
-    uploads = []
-    uploads_archs = []
-    uploads_closes = []
     for name in glob(path + '/debian-devel-changes.*'):
-      # print name
       f = None
       if name.endswith(".gz"):
         f = gzip.open(name)
       else:
-            f = open(name)
+        f = open(name)
       current = {'id': id}
       current['Fingerprint'] = 'N/A' # hack: some entries don't have fp
       last_field = None
       line_count = 0
+      uploads = []
+      uploads_archs = []
+      uploads_closes = []
 
       for line in f:
         line_count += 1
@@ -101,10 +100,11 @@
         current[field] = data
         
         last_field = field
+
+      cursor.executemany(query, uploads)
+      cursor.executemany(query_archs, uploads_archs)
+      cursor.executemany(query_closes, uploads_closes)
       
-    cursor.executemany(query, uploads)
-    cursor.executemany(query_archs, uploads_archs)
-    cursor.executemany(query_closes, uploads_closes)
     cursor.execute("DEALLOCATE uh_insert")
     cursor.execute("ANALYZE " + self.my_config['table'] + '_architecture')
     cursor.execute("ANALYZE " + self.my_config['table'] + '_closes')