From b38a688cf7e3b0ed8dfa917b4d4ab14b0c44342a Mon Sep 17 00:00:00 2001 From: epriestley Date: Fri, 13 Sep 2013 09:38:42 -0700 Subject: [PATCH] Ignore failures when rebuilding project search index in Maniphest migration Summary: See . Just ignore any issues here, they aren't worth complaining about. Auditors: btrahan --- resources/sql/patches/20130912.maniphest.4.fillindex.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/resources/sql/patches/20130912.maniphest.4.fillindex.php b/resources/sql/patches/20130912.maniphest.4.fillindex.php index a1c6738a03..d4d4a4db61 100644 --- a/resources/sql/patches/20130912.maniphest.4.fillindex.php +++ b/resources/sql/patches/20130912.maniphest.4.fillindex.php @@ -12,5 +12,9 @@ $command = new PhutilExecPassthru( $root.'/scripts/search/manage_search.php'); $err = $command->execute(); if ($err) { - throw new Exception("Update failed!"); + // 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. }