Paths

Table of Contentst

Differential D19502

When queueing search index tasks, include the "objectPHID" in the task metadata
ClosedPublic
Actions

Authored by epriestley on Jun 22 2018, 12:40 PM.

Tags

None

Referenced Files

	F18185779: D19502.id46639.diff
	Sat, Aug 16, 4:59 PM

	F18110699: D19502.id46633.diff
	Mon, Aug 11, 10:35 PM

	F18105165: D19502.id.diff
	Sun, Aug 10, 4:40 PM

	F18101382: D19502.diff
	Sat, Aug 9, 12:45 PM

	Unknown Object (File)
	May 7 2025, 4:47 AM

	Unknown Object (File)
	Apr 24 2025, 6:48 PM

	Unknown Object (File)
	Apr 24 2025, 3:15 PM

	Unknown Object (File)
	Apr 21 2025, 8:51 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13151: Plans: 2018 Week 23 - Week 30 Bonus Content

Commits

rPcbc71e75fae3: When queueing search index tasks, include the "objectPHID" in the task metadata

Summary

Ref T13151. See PHI719. One minor hiccup in debugging the issue (which ended up being "revision has 100K comments") was that the SearchWorker did not show which object it was indexing.

Add 'objectPHID' to the queue call so you can see which object is affected from the web UI.

Test Plan

Stopped daemons.
Used bin/search index D123 --background to queue a search task.
Viewed task details in web UI from /daemon/.
- Before change: no indication of which object was being indexed.
- After change: page helpfully shows that the task is indexing D123.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 22 2018, 12:40 PM

Harbormaster completed remote builds in B20439: Diff 46633.Jun 22 2018, 12:41 PM

epriestley requested review of this revision.Jun 22 2018, 12:41 PM

epriestley added a child revision: D19503: Index only the first 1,000 comments on any object.Jun 22 2018, 4:10 PM

amckinley accepted this revision.Jun 22 2018, 11:17 PM

This revision is now accepted and ready to land.Jun 22 2018, 11:17 PM

Closed by commit rPcbc71e75fae3: When queueing search index tasks, include the "objectPHID" in the task metadata (authored by epriestley). · Explain WhyJun 23 2018, 12:40 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

search/

worker/

PhabricatorSearchWorker.php

1 line

Diff 46639

src/applications/search/worker/PhabricatorSearchWorker.php

Loading...