Page MenuHomePhabricator

Allow `*.search` Conduit API methods to have data bulk-loaded by extensions
ClosedPublic

Authored by epriestley on Jul 31 2016, 3:29 PM.
Tags
None
Referenced Files
F18663447: D16350.id.diff
Tue, Sep 23, 10:39 PM
F18629389: D16350.diff
Tue, Sep 16, 8:39 AM
F18600792: D16350.id39319.diff
Sat, Sep 13, 10:36 AM
F18507940: D16350.id.diff
Fri, Sep 5, 2:31 AM
F18182489: D16350.id39327.diff
Aug 16 2025, 9:44 AM
F18104949: D16350.id39319.diff
Aug 10 2025, 3:15 PM
F18103553: D16350.id.diff
Aug 10 2025, 6:22 AM
F18101961: D16350.diff
Aug 9 2025, 5:05 PM
Subscribers
None

Details

Summary

Ref T11404. Currently, SearchEngineAttachments can bulk-load data but SearchEngineExtensions can not.

This leads to poor performance of custom fields. See T11404 for discussion.

This changes the API to support a bulk load + format pattern like the one Attachments use. The next change will use it to bulk-load custom field data.

Test Plan
  • Ran differential.query, differential.revision.search as a sanity check.
  • No behavioral changes are expected
  • See next revision.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow `*.search` Conduit API methods to have data bulk-loaded by extensions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, yelirekim.
chad edited edge metadata.
This revision is now accepted and ready to land.Jul 31 2016, 3:37 PM
This revision was automatically updated to reflect the committed changes.