Skip to content

search: more consistently report timeout errors instead of "No results" when `timeout:` directive is present

Warren Gifford requested to merge 3294-indexed-search-timeouts into master

Created by: ijt

This is progress on #3294 (closed) .

This also includes a helpful hint for a longer timeout directive in the error message.

Test plan: Manually tested with queries like func timeout:1ms on a sourcegraph instance with a single repo

Before: Screen Shot 2019-04-12 at 4 55 25 PM

After: Screen Shot 2019-04-12 at 4 42 37 PM

The commits shown on github are screwed up and include things from a previous PR. Sorry about that.

Merge request reports

Loading