From 543a3133877ecb4f038621b5f4133c35b70f598d Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 26 Mar 2014 13:51:49 -0700 Subject: [PATCH] Skip a very old project reindex migration Summary: During migration of very old installs, this script no longer runs properly since at HEAD it can't index against older schemas. Since it's pretty fluff, just toss it. Installs can run `bin/search index --type PROJ` after finishing migrations to achieve the same effect, if necessary. Test Plan: eyeballed it Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D8619 --- .../20130912.maniphest.4.fillindex.php | 21 +++---------------- 1 file changed, 3 insertions(+), 18 deletions(-) diff --git a/resources/sql/patches/20130912.maniphest.4.fillindex.php b/resources/sql/patches/20130912.maniphest.4.fillindex.php index d4d4a4db61..4dfaff8eb4 100644 --- a/resources/sql/patches/20130912.maniphest.4.fillindex.php +++ b/resources/sql/patches/20130912.maniphest.4.fillindex.php @@ -1,20 +1,5 @@ execute(); -if ($err) { - // NOTE: Just ignore this. The indexing script fails if there are no projects - // yet, and even if it actually fails with something terrible and explosive - // it isn't very concerning. The worst case here is that "group by projects" - // doesn't work perfectly in Maniphest, and re-running the script later on - // can fix it. -} +// This previously updated the PROJ index, but HEAD can no longer reindex +// old PROJ records properly (since fields have been added to various places) +// and this migration is 6 months old and optional. Just skip it. -- 2.51.2