internal/postgres: remove some error logging We were logging some information to debug a deadlock, but it was cluttering the logs and the messages had no context viewed in isolation. Instead just log one row. Change-Id: I0c7c261af0371f12edf51102a0b6ec20daf4bc9b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/348932 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jamal Carvalho <jamal@golang.org>
diff --git a/internal/postgres/insert_module.go b/internal/postgres/insert_module.go index c253e99..7c8c08e 100644 --- a/internal/postgres/insert_module.go +++ b/internal/postgres/insert_module.go
@@ -497,10 +497,7 @@ pathIDToUnitID[pathID] = unitID return nil }); err != nil { - log.Errorf(ctx, "got error doing bulk upsert to units (see below); logging path_id, module_id for golang.org/issue/43899") - for i := 0; i < len(unitValues); i += len(unitCols) { - log.Errorf(ctx, "%v, %v", unitValues[i], unitValues[i+1]) - } + log.Errorf(ctx, "got error doing bulk upsert to units (see below); logging one path_id, module_id for golang.org/issue/43899: %v, %v", unitValues[0], unitValues[1]) return nil, err } return pathIDToUnitID, nil