dbworker: include recordID in fetchDebugInformationForJob error (#58036)

keegancsmith · vovakulikov · commit 9aa98911e411 · 2023-12-12T18:48:01.000-03:00
Not sure why go's format string linter isn't picking this up.

Test Plan: n/a
diff --git a/internal/workerutil/dbworker/store/store.go b/internal/workerutil/dbworker/store/store.go
@@ -1056,7 +1056,7 @@ func (s *store[T]) fetchDebugInformationForJob(ctx context.Context, recordID int
 		return "", err
 	}
 	if !ok {
-		return "", errors.Newf("fetching debug information for record %d didn't return rows")
+		return "", errors.Newf("fetching debug information for record %d didn't return rows", recordID)
 	}
 	return debug, nil
 }

Original file line number	Diff line number	Diff line change
`@@ -1056,7 +1056,7 @@ func (s *store[T]) fetchDebugInformationForJob(ctx context.Context, recordID int`
`1056`	`1056`	`return "", err`
`1057`	`1057`	`}`
`1058`	`1058`	`if !ok {`
`1059`		`- return "", errors.Newf("fetching debug information for record %d didn't return rows")`
	`1059`	`+ return "", errors.Newf("fetching debug information for record %d didn't return rows", recordID)`
`1060`	`1060`	`}`
`1061`	`1061`	`return debug, nil`
`1062`	`1062`	`}`