diff options
| author | Julie Qiu <julie@golang.org> | 2020-11-04 11:33:49 -0500 |
|---|---|---|
| committer | Julie Qiu <julie@golang.org> | 2020-11-04 16:50:23 +0000 |
| commit | 132a1fac40cfea1ac0b428fa2fd5b0a641ad7ef7 (patch) | |
| tree | 17189e1fac3ba1d9c28058f090ac7aa9cc052baf /internal/postgres/insert_module.go | |
| parent | b99e07b928a1cecf75ca1782d36e69d948310d85 (diff) | |
| download | go-x-pkgsite-132a1fac40cfea1ac0b428fa2fd5b0a641ad7ef7.tar.xz | |
internal/postgres: delete unused logMemory and allocMeg
Change-Id: I17b56df6d85db40220351017a450b0e5c0cd500c
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/267679
Reviewed-by: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Trust: Julie Qiu <julie@golang.org>
Diffstat (limited to 'internal/postgres/insert_module.go')
| -rw-r--r-- | internal/postgres/insert_module.go | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/internal/postgres/insert_module.go b/internal/postgres/insert_module.go index 36d028b9..f65b4944 100644 --- a/internal/postgres/insert_module.go +++ b/internal/postgres/insert_module.go @@ -12,7 +12,6 @@ import ( "fmt" "hash/fnv" "io" - "runtime" "sort" "strings" "unicode/utf8" @@ -77,23 +76,17 @@ func (db *DB) saveModule(ctx context.Context, m *internal.Module) (err error) { ctx, span := trace.StartSpan(ctx, "saveModule") defer span.End() - logMemory(ctx, "at start of saveModule") return db.db.Transact(ctx, sql.LevelDefault, func(tx *database.DB) error { moduleID, err := insertModule(ctx, tx, m) if err != nil { return err } - logMemory(ctx, "after insertModule") - if err := insertLicenses(ctx, tx, m, moduleID); err != nil { return err } - logMemory(ctx, "after insertLicenses") - if err := db.insertUnits(ctx, tx, m, moduleID); err != nil { return err } - logMemory(ctx, "after insertUnits") // Obtain a transaction-scoped exclusive advisory lock on the module // path. The transaction that holds the lock is the only one that can @@ -640,17 +633,3 @@ func makeValidUnicode(s string) string { } return b.String() } - -var MemoryLoggingDisabled = true - -func logMemory(ctx context.Context, msg string) { - if !MemoryLoggingDisabled { - log.Debugf(ctx, "memory %s: %dM", msg, allocMeg()) - } -} - -func allocMeg() int { - var ms runtime.MemStats - runtime.ReadMemStats(&ms) - return int(ms.Alloc / (1024 * 1024)) -} |
