tree 5159a945e1154c2e64be3b32fd6377900fe644aa
parent fc98402a2851aae217ef2c5071e8a518f6f999c1
author Julie Qiu <julie@golang.org> 1629131606 -0400
committer Julie Qiu <julie@golang.org> 1629225546 +0000

internal/postgres/{symbolsearch}: update queries

The symbolsearch queries are rewritten to filter on
symbol_search_documents.symbol_name, instead of symbols.name or
symbols.tsv_name_tokens.

This significantly improves performance on the first search because:

(1) We no longer need to fetch symbol IDs first, before performing the
    search query on symbol_search_documents, which can save ~100-200ms.
(2) Searching on the lower(symbol_name) index is faster than the
    symbol_name_id index. This likely because the lower(symbol_name) =
    lower($1) fetches rows that are paged together, whereas the
    symbol_name_ids maybe completely random.

For golang/go#44142

Change-Id: Ibe78676d4f5424dcace9da544f46a6445a06b160
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/342471
Trust: Julie Qiu <julie@golang.org>
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>