Bug 1071577 - Moved result_set revision hash cache step to after asso…

…ciated revision storage
mozilla · Sep 24, 2014 · 2c258f4 · 2c258f4
1 parent d790879
commit 2c258f4
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/treeherder/model/derived/jobs.py b/treeherder/model/derived/jobs.py
@@ -2601,11 +2601,6 @@ def store_result_set_data(self, result_sets):
 
                 cache_data[key] = revision_hash
 
-            cache.set_many(
-                cache_data,
-                self.RESULT_SET_CACHE_TIMEOUT
-                )
-
             # Insert new revisions
             dhub.execute(
                 proc='jobs.inserts.set_revision',
@@ -2614,6 +2609,11 @@ def store_result_set_data(self, result_sets):
                 debug_show=self.DEBUG
                 )
 
+            cache.set_many(
+                cache_data,
+                self.RESULT_SET_CACHE_TIMEOUT
+                )
+
         # Retrieve new revision ids
         rev_where_in_clause = ','.join(rev_where_in_list)
         select_proc = 'get_revision_ids'