phorge-phorge/resources/sql/autopatches/20160928.repo.messagecount.sql at da1ed2c63ac9b45c34fec4f14c178bd4f2a9fc0f - revi-archive/phorge-phorge - SiliconForest Atelier

revi-archive/phorge-phorge

mirror of https://we.phorge.it/source/phorge.git synced 2024-12-20 20:40:56 +01:00

epriestley 5d1359d78f Fix an issue where repository message counts would never reset

Summary:
Fixes T11705. I did not realize that `ON DUPLICATE KEY UPDATE` was order-dependent, so the "reset" clause of this `IF(...)` never actually worked.

Reorder it so we check if we're changing the message type //first//, then actually change the message type.

This makes the count reset properly when a failing repository succeeds, or a working repository fails.

Test Plan:
  - On `master`, forced a working repository to fail a `bin/repository update`, saw the message change types (expected) but keep the old count (wrong!).
  - With this patch, repeated the process and saw the count reset properly.
  - Ran the patch, verified counts reset to 0.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11705

Differential Revision: https://secure.phabricator.com/D16623

2016-09-28 15:02:26 -07:00

4 lines

183 B

SQL

Raw Blame History

 /* Reset message counts to fix the bug in T11705 which caused some of them to
    become very large. */
 UPDATE {$NAMESPACE}_repository.repository_statusmessage
   SET messageCount = 0;