Page MenuHomePhabricator

Perform search indexing in the worker queue and respect `bin/search index --background`
ClosedPublic

Authored by epriestley on Jan 14 2014, 8:31 PM.
Tags
None
Referenced Files
F14032902: D7966.diff
Sat, Nov 9, 4:09 PM
F14021493: D7966.diff
Wed, Nov 6, 9:34 AM
F14006724: D7966.diff
Mon, Oct 28, 3:48 PM
F13993381: D7966.id18017.diff
Tue, Oct 22, 10:30 PM
F13993182: D7966.diff
Tue, Oct 22, 9:13 PM
F13990407: D7966.id18018.diff
Tue, Oct 22, 3:27 AM
F13989927: D7966.id18018.diff
Mon, Oct 21, 11:58 PM
F13989639: D7966.diff
Mon, Oct 21, 9:57 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPa716fe99f3b2: Perform search indexing in the worker queue and respect `bin/search index…
Summary

Fixes T3857. Earlier work made this trivial and just left product questions, which I've answered by requiring the daemons to run on reasonable installs.

Test Plan

Ran bin/search index and bin/search index --background. Observed indexes write in the former case and tasks queue in the latter case. Commented with a unique string on a revision and searched for it a moment later, got exactly one result (that revision), verifying that reindexing works correctly.

Diff Detail

Branch
auth7
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

btrahan added inline comments.
src/applications/search/management/PhabricatorSearchManagementIndexWorkflow.php
32

the it sentence is hard to read

epriestley updated this revision to Unknown Object (????).Jan 14 2014, 9:22 PM
  • Make it the it easier to read it.