[llvm-commits] [zorg] r106241 - /zorg/trunk/lnt/lnt/db/perfdbsummary.py

[Daniel Dunbar]

Author: ddunbar
Date: Thu Jun 17 16:43:17 2010
New Revision: 106241

URL: http://llvm.org/viewvc/llvm-project?rev=106241&view=rev
Log:
LNT: Fix a caching bug where we didn't properly invalidate summary objects when
fetching from a database which had a pending uncommitted transaction.

Modified:
    zorg/trunk/lnt/lnt/db/perfdbsummary.py

Modified: zorg/trunk/lnt/lnt/db/perfdbsummary.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/lnt/lnt/db/perfdbsummary.py?rev=106241&r1=106240&r2=106241&view=diff
==============================================================================
--- zorg/trunk/lnt/lnt/db/perfdbsummary.py (original)
+++ zorg/trunk/lnt/lnt/db/perfdbsummary.py Thu Jun 17 16:43:17 2010
@@ -33,7 +33,8 @@
         self.suites = suites
 
     def is_up_to_date(self, db):
-        return self.revision == db.get_revision_number("Run")
+        return (not db.modified_run and
+                self.revision == db.get_revision_number("Run"))
 
 class SimpleSuiteSummary(object):
     @staticmethod
@@ -112,7 +113,8 @@
         self.parameter_sets = parameter_sets
 
     def is_up_to_date(self, db):
-        return self.revision == db.get_revision_number("Test")
+        return (not db.modified_test and
+                self.revision == db.get_revision_number("Test"))
 
 _cache = {}
 def get_simple_suite_summary(db, tag):
@@ -186,7 +188,8 @@
         self.machine_id_by_run = machine_id_by_run
 
     def is_up_to_date(self, db):
-        return self.revision == db.get_revision_number("RunInfo")
+        return (not db.modified_run and
+                self.revision == db.get_revision_number("RunInfo"))
 
     def get_run_order(self, run_id):
         return self.order_by_run.get(run_id)