Paths

Table of Contentst

Differential D14174

Refine error behavior of `bin/search index`
ClosedPublic
Actions

Authored by epriestley on Sep 27 2015, 5:50 PM.

Tags

None

Referenced Files

	F19149346: D14174.id34246.diff
	Wed, Dec 10, 7:45 PM

	F19148740: D14174.id34246.diff
	Wed, Dec 10, 6:45 PM

	F19078802: D14174.id34244.diff
	Mon, Dec 1, 7:27 PM

	F19059213: D14174.id34244.diff
	Sat, Nov 29, 4:50 AM

	F18851415: D14174.id.diff
	Oct 31 2025, 12:55 AM

	F18809482: D14174.diff
	Oct 19 2025, 2:23 PM

	F18798812: D14174.id.diff
	Oct 17 2025, 8:56 AM

	F18793067: D14174.diff
	Oct 16 2025, 8:41 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T5991: `bin/search index` returns 0 on failure

Commits

Restricted Diffusion Commit
rP24845c70b918: Refine error behavior of `bin/search index`

Summary

Fixes T5991. If all requested documents failed to index, consider this a catastrophic failure and exit with an error code.

Test Plan

Ran bin/search index --type TASK, observed successful exit despite a small number of un-indexable documents.
Ran bin/search index PHID-TASK-xxx for an invalid task, observed exception on exit after complete failure.
Ran normal indexing through daemons.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 34244.Sep 27 2015, 5:50 PM

epriestley retitled this revision from to Refine error behavior of `bin/search index`.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T5991: `bin/search index` returns 0 on failure.

epriestley mentioned this in T5991: `bin/search index` returns 0 on failure.Sep 27 2015, 5:54 PM

chad accepted this revision.Sep 27 2015, 6:52 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Sep 27 2015, 6:52 PM

Closed by commit rP24845c70b918: Refine error behavior of `bin/search index` (authored by epriestley, committed by epriestley). · Explain WhySep 27 2015, 8:11 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

search/

index/

PhabricatorSearchDocumentIndexer.php

64 lines

management/

PhabricatorSearchManagementIndexWorkflow.php

15 lines

worker/

PhabricatorSearchWorker.php

12 lines

Diff 34246

src/applications/search/index/PhabricatorSearchDocumentIndexer.php

Loading...

src/applications/search/management/PhabricatorSearchManagementIndexWorkflow.php

Loading...

src/applications/search/worker/PhabricatorSearchWorker.php

Loading...