internal/postgres: invoke middleware.ElapsedStat
middleware.ElapsedStat wasn't being called on GetVersionsForPath and
GetPackageSymbols, which is now fixed.
Change-Id: Ie6b9003eca509448a4bb2e5aa127ca1f32777ab5
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/311929
Trust: Julie Qiu <julie@golang.org>
Run-TryBot: Julie Qiu <julie@golang.org>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
diff --git a/internal/postgres/symbol_history.go b/internal/postgres/symbol_history.go
index 77788aa..f8e85fe 100644
--- a/internal/postgres/symbol_history.go
+++ b/internal/postgres/symbol_history.go
@@ -19,7 +19,7 @@
func (db *DB) GetPackageSymbols(ctx context.Context, packagePath, modulePath string,
) (_ map[string]map[string]*internal.UnitSymbol, err error) {
defer derrors.Wrap(&err, "GetPackageSymbols(ctx, db, %q, %q)", packagePath, modulePath)
- defer middleware.ElapsedStat(ctx, "GetPackageSymbols")
+ defer middleware.ElapsedStat(ctx, "GetPackageSymbols")()
doctable := "new_documentation"
if experiment.IsActive(ctx, internal.ExperimentDoNotInsertNewDocumentation) {
diff --git a/internal/postgres/version.go b/internal/postgres/version.go
index d9d16ea..4cd8388 100644
--- a/internal/postgres/version.go
+++ b/internal/postgres/version.go
@@ -28,7 +28,7 @@
// recent from a list of pseudo-versions sorted in descending semver order.
func (db *DB) GetVersionsForPath(ctx context.Context, path string) (_ []*internal.ModuleInfo, err error) {
defer derrors.WrapStack(&err, "GetVersionsForPath(ctx, %q)", path)
- defer middleware.ElapsedStat(ctx, "GetVersionsForPath")
+ defer middleware.ElapsedStat(ctx, "GetVersionsForPath")()
versions, err := getPathVersions(ctx, db, path, version.TypeRelease, version.TypePrerelease)
if err != nil {
