[LNT] r311950 - Fix MySQL funnyness where test names are not in the cache

Chris Matthews via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 28 16:07:15 PDT 2017


Author: cmatthews
Date: Mon Aug 28 16:07:15 2017
New Revision: 311950

URL: http://llvm.org/viewvc/llvm-project?rev=311950&view=rev
Log:
Fix MySQL funnyness where test names are not in the cache

Seems like this cache is failing, and we get integrity errors. In
David's original patch he downcasted here to make it match, so I will
do the same.

Modified:
    lnt/trunk/lnt/server/db/testsuitedb.py

Modified: lnt/trunk/lnt/server/db/testsuitedb.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/server/db/testsuitedb.py?rev=311950&r1=311949&r2=311950&view=diff
==============================================================================
--- lnt/trunk/lnt/server/db/testsuitedb.py (original)
+++ lnt/trunk/lnt/server/db/testsuitedb.py Mon Aug 28 16:07:15 2017
@@ -974,7 +974,8 @@ class TestSuiteDB(object):
     def _importSampleValues(self, tests_data, run, commit, config):
         # Load a map of all the tests, which we will extend when we find tests
         # that need to be added.
-        test_cache = dict((test.name, test)
+        # Downcast to str, so we match on MySQL.
+        test_cache = dict((str(test.name), test)
                           for test in self.query(self.Test))
 
         profiles = dict()




More information about the llvm-commits mailing list