internal/postgres/symbolsearch: update queries
The symbol search queries are updated to ORDER BY imported_by_count and
use the new idx_symbol_search_documents_symbol_name_imported_by_count
index.
symbol_name_id is removed as a sort clause, since it isn't possible for
a single package to have more than 1 result per symbol, now that we
don't allow searching by just field or method names.
For golang/go#44142
Change-Id: Ifd061b0fbc5e4c4efccebe6de5d99038a3db1f17
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/343653
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>
diff --git a/internal/postgres/symbolsearch/query.gen.go b/internal/postgres/symbolsearch/query.gen.go
index ee038ae..d387b45 100644
--- a/internal/postgres/symbolsearch/query.gen.go
+++ b/internal/postgres/symbolsearch/query.gen.go
@@ -17,14 +17,13 @@
ssd.symbol_name_id,
ssd.goos,
ssd.goarch,
- ssd.ln_imported_by_count AS score
+ ssd.imported_by_count AS score
FROM symbol_search_documents ssd
WHERE
lower(symbol_name) = lower($1)
ORDER BY
score DESC,
- package_path,
- symbol_name_id
+ package_path
LIMIT $2
)
SELECT
@@ -58,7 +57,7 @@
ssd.symbol_name_id,
ssd.goos,
ssd.goarch,
- ssd.ln_imported_by_count AS score
+ ssd.imported_by_count AS score
FROM symbol_search_documents ssd
WHERE
lower(symbol_name) = lower($1)
@@ -68,8 +67,7 @@
)
ORDER BY
score DESC,
- package_path,
- symbol_name_id
+ package_path
LIMIT $2
)
SELECT
diff --git a/internal/postgres/symbolsearch/symbolsearch.go b/internal/postgres/symbolsearch/symbolsearch.go
index 70b9f64..c3fe344 100644
--- a/internal/postgres/symbolsearch/symbolsearch.go
+++ b/internal/postgres/symbolsearch/symbolsearch.go
@@ -49,13 +49,12 @@
ssd.symbol_name_id,
ssd.goos,
ssd.goarch,
- ssd.ln_imported_by_count AS score
+ ssd.imported_by_count AS score
FROM symbol_search_documents ssd
WHERE %s
ORDER BY
score DESC,
- package_path,
- symbol_name_id
+ package_path
LIMIT $2
`