Page MenuHomePhabricator

Refine error behavior of `bin/search index`
ClosedPublic

Authored by epriestley on Sep 27 2015, 5:50 PM.
Tags
None
Referenced Files
F19675413: D14174.id34244.diff
Sun, Feb 8, 9:37 AM
F19675404: D14174.id34244.diff
Sun, Feb 8, 9:32 AM
F19609637: D14174.id.diff
Tue, Feb 3, 1:45 PM
F19583259: D14174.id34244.diff
Mon, Feb 2, 12:00 AM
F19580219: D14174.id34244.diff
Sun, Feb 1, 5:19 PM
F19562298: D14174.id34244.diff
Sat, Jan 31, 12:22 PM
F19554851: D14174.id34244.diff
Fri, Jan 30, 4:21 AM
F19149346: D14174.id34246.diff
Dec 10 2025, 7:45 PM
Subscribers
None

Details

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
rP Phabricator
Branch
xcode2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8069
Build 9170: [Placeholder Plan] Wait for 30 Seconds
Build 9169: arc lint + arc unit

Event Timeline

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.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 27 2015, 6:52 PM
This revision was automatically updated to reflect the committed changes.