koji

#2635 update image volume in db ASAP
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue2634  into  master

Download 2635.patch

Fixes: https://pagure.io/koji/issue/2634

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done

rebased onto 234ced2838c1f7241691627a98ec5a517ea385c2

Commit 548cb4a0 fixes this pull-request

Pull-Request has been merged by tkopecek

Side note: this changes introduces a double update to the same row in a single transaction.

While there should be nothing wrong with such a thing, we have avoided this in Koji because of problems with Postgres deadlocks in the past. At least at one point, postgres locking behavior for could be thrown off by double updates like this, leading to deadlocks. However, I'm not sure if that is still the case. I'll file a follow-up ticket to remind us to investigate.
Metadata
None

testing-done testing-ready

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.