[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