[llvm-commits] [zorg] r147539 - /zorg/trunk/lnt/lnt/server/db/v4db.py

[Daniel Dunbar]

Author: ddunbar
Date: Wed Jan  4 12:33:56 2012
New Revision: 147539

URL: http://llvm.org/viewvc/llvm-project?rev=147539&view=rev
Log:
[lnt/v0.4] lnt.server.db.v4db: Don't let the summary object potentially reuse
the database cursor which may have come from another thread. This isn't the
right fix, but this class is slated for termination anyway.

Modified:
    zorg/trunk/lnt/lnt/server/db/v4db.py

Modified: zorg/trunk/lnt/lnt/server/db/v4db.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/lnt/lnt/server/db/v4db.py?rev=147539&r1=147538&r2=147539&view=diff
==============================================================================
--- zorg/trunk/lnt/lnt/server/db/v4db.py (original)
+++ zorg/trunk/lnt/lnt/server/db/v4db.py Wed Jan  4 12:33:56 2012
@@ -136,10 +136,15 @@
 
     def __init__(self, db):
         self.db = db
+        # Load all the test suite names now so that we don't attempt to reuse a
+        # cursor later.
+        #
+        # FIXME: Really, we just need to eliminate this object.
+        self.testsuites = list(self.db.testsuite)
 
     @property
     def suites(self):
-        for name in self.db.testsuite:
+        for name in self.testsuites:
             yield V4DBSummary.SuiteSummary(name, ("v4", name))
 
     def is_up_to_date(self, db):