From 132a1fac40cfea1ac0b428fa2fd5b0a641ad7ef7 Mon Sep 17 00:00:00 2001 From: Julie Qiu Date: Wed, 4 Nov 2020 11:33:49 -0500 Subject: internal/postgres: delete unused logMemory and allocMeg Change-Id: I17b56df6d85db40220351017a450b0e5c0cd500c Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/267679 Reviewed-by: Jonathan Amsterdam Run-TryBot: Julie Qiu TryBot-Result: kokoro Trust: Julie Qiu --- internal/postgres/insert_module.go | 21 --------------------- 1 file changed, 21 deletions(-) (limited to 'internal/postgres/insert_module.go') 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)) -} -- cgit v1.3-5-g9baa