D8619.id.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D8619.id.diff
View Options

	diff --git a/resources/sql/patches/20130912.maniphest.4.fillindex.php b/resources/sql/patches/20130912.maniphest.4.fillindex.php
	--- a/resources/sql/patches/20130912.maniphest.4.fillindex.php
	+++ b/resources/sql/patches/20130912.maniphest.4.fillindex.php
	@@ -1,20 +1,5 @@
	<?php

	-// Update the "PROJ" search index, to:
	-//
	-// - Populate the index itself, which was added recently.
	-// - Populate the secondary object name index in Maniphest.
	-
	-$root = dirname(phutil_get_library_root('phabricator'));
	-
	-$command = new PhutilExecPassthru(
	- 'php -f %s -- index --type PROJ',
	- $root.'/scripts/search/manage_search.php');
	-$err = $command->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.