internal/etl: add bug for UpsertVersionState split TODO
@@ -102,8 +102,8 @@ func fetchAndUpdateState(ctx context.Context, modulePath, version string, client
// module@version. This must happen last, because if it succeeds with a
// code < 500 but a later action fails, we will never retry the later action.
- // TODO(jba): Split UpsertVersionState into InsertVersionState and UpdateVersionState.
- // See if err := db.UpsertVersionState(ctx, modulePath, version, appVersionLabel, time.Time{}, code, fetchErr); err != nil {
+ // TODO(b/139178863): Split UpsertVersionState into InsertVersionState and UpdateVersionState.
+ if err := db.UpsertVersionState(ctx, modulePath, version, appVersionLabel, time.Time{}, code, fetchErr); err != nil {
log.Printf("db.UpsertVersionState(ctx, %q, %q, %q, %q, %v): %q", modulePath, version, config.AppVersionLabel(), code, fetchErr, err)
if fetchErr != nil {
err = fmt.Errorf("error updating version state: %v, original error: %v", err, fetchErr)
