diff options
| author | Chocobozzz <me@florianbigard.com> | 2021-04-09 16:29:38 +0200 |
|---|---|---|
| committer | Chocobozzz <me@florianbigard.com> | 2021-04-09 16:29:38 +0200 |
| commit | 78ec174cca0f03a0647b0d39c8d91082f63d98db (patch) | |
| tree | 8b2e0e89182a112e7a35960e197f184157af121c | |
| parent | 21c917b38bcb2eb3e541f745a0a6d8456e3d57b1 (diff) | |
| download | PeerTube-78ec174cca0f03a0647b0d39c8d91082f63d98db.tar.gz PeerTube-78ec174cca0f03a0647b0d39c8d91082f63d98db.tar.zst PeerTube-78ec174cca0f03a0647b0d39c8d91082f63d98db.zip | |
Don't retry failed transaction for refresh
It's just a refresh and could lead to some issues in our tests. If there
is a transaction conflict, it means that the same video is already
beeing updated so aborting should be fine
| -rw-r--r-- | server/lib/activitypub/videos.ts | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/server/lib/activitypub/videos.ts b/server/lib/activitypub/videos.ts index 9014791c0..506204674 100644 --- a/server/lib/activitypub/videos.ts +++ b/server/lib/activitypub/videos.ts | |||
| @@ -539,7 +539,7 @@ async function refreshVideoIfNeeded (options: { | |||
| 539 | account: channelActor.VideoChannel.Account, | 539 | account: channelActor.VideoChannel.Account, |
| 540 | channel: channelActor.VideoChannel | 540 | channel: channelActor.VideoChannel |
| 541 | } | 541 | } |
| 542 | await retryTransactionWrapper(updateVideoFromAP, updateOptions) | 542 | await updateVideoFromAP(updateOptions) |
| 543 | await syncVideoExternalAttributes(video, videoObject, options.syncParam) | 543 | await syncVideoExternalAttributes(video, videoObject, options.syncParam) |
| 544 | 544 | ||
| 545 | ActorFollowScoreCache.Instance.addGoodServerId(video.VideoChannel.Actor.serverId) | 545 | ActorFollowScoreCache.Instance.addGoodServerId(video.VideoChannel.Actor.serverId) |