Page MenuHomePhabricator

By default, do not save queries when executing Conduit "*.search" calls
ClosedPublic

Authored by epriestley on Jul 9 2016, 1:50 PM.
Tags
None
Referenced Files
F13995914: D16263.id.diff
Wed, Oct 23, 4:40 PM
F13991626: D16263.id39123.diff
Tue, Oct 22, 11:52 AM
F13959445: D16263.diff
Oct 14 2024, 7:27 PM
Unknown Object (File)
Sep 21 2024, 7:15 AM
Unknown Object (File)
Sep 1 2024, 10:04 PM
Unknown Object (File)
Aug 29 2024, 12:47 AM
Unknown Object (File)
Aug 20 2024, 3:33 AM
Unknown Object (File)
Aug 19 2024, 2:48 AM
Subscribers
None

Details

Summary

Fixes T11304. Prior to this change, we did an unnecessary write on every "*.search" call (this write didn't always actually write a row, since we only save unique saved queries, but still doesn't do anything useful ever, currently).

Instead, change this to not-write by default. We could add an "oh, and also I want you to do a write" option later, which would let us implement something like arc query-stuff which says "To see more results, view this URI in your browser: ...".

(It's possible to run one of these methods with an existing SavedQuery by using the key, so we still sometimes have a queryKey to return.)

Test Plan

Ran almanac.service.search, used DarkConsole to verify that no serachengine writes occurred.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to By default, do not save queries when executing Conduit "*.search" calls.
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.Jul 9 2016, 5:24 PM
This revision was automatically updated to reflect the committed changes.