[PATCH] D34586: Remove info_keys; NFC

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 16:05:50 PDT 2017


MatzeB added inline comments.


================
Comment at: lnt/server/db/migrations/upgrade_12_to_13.py:13
+        session.connection().execute("""
+ALTER TABLE TestSuiteMachineFields DROP COLUMN info_key;
+ALTER TABLE TestSuiteOrderFields DROP COLUMN info_key;
----------------
cmatthews wrote:
> Could you land the DROPing part of the much later, once this has made it into the wild and everyone is happy.  There is no revert from a DROP!
Indeed I had the same idea. I was thinking about waiting until we have an upgraded server that was running fine for a week or two...


Repository:
  rL LLVM

https://reviews.llvm.org/D34586





More information about the llvm-commits mailing list