tree 7777917125ccdf58762fae723091f9358687ded9
parent 5154d97bc474524e1c219d0e0428c826acabc488
author Jonathan Amsterdam <jba@google.com> 1627043591 -0400
committer Jonathan Amsterdam <jba@google.com> 1627071958 +0000

internal/postgres: repeat search when grouping

When grouping results, search a second time with a larger limit if the
first doesn't return enough results.

Also, remove the Limit search option; it is an implementation detail
that should be determined by DB.Search.

I tested this against 176 distinct queries obtained from the logs over
a 1-hour period, with max results = 10. Although it was somewhat
slower than without grouping, the 99%ile was still under 1 second.

For golang/go#47320

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