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
F19019613: D16263.id.diff
Nov 23 2025, 3:26 PM
F19010736: D16263.diff
Nov 22 2025, 6:19 AM
F18853717: D16263.id.diff
Oct 31 2025, 6:58 PM
F18849260: D16263.id39132.diff
Oct 30 2025, 8:47 AM
F18821834: D16263.id.diff
Oct 23 2025, 2:00 AM
F18819820: D16263.diff
Oct 22 2025, 10:00 AM
F18807645: D16263.id39132.diff
Oct 19 2025, 2:41 AM
F18651032: D16263.diff
Sep 21 2025, 2:38 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.