Badge grant fails if any throw exception


(Kane York) #1

Continuing the discussion from [solved] Badges 'update daily' won't trigger:


[solved] Badges 'update daily' won't trigger
(Kane York) #2

@sam it’s bad that one rogue query can stop all the daily badges, people should get a /admin alert for this


(Sam Saffron) #3

Sure PR to fix this welcome, we should also be logging warnings for any “delta” query that takes longer than 200ms and daily that takes longer than 2 seconds (configurable in site settings)


(Kane York) #4

https://github.com/discourse/discourse/pull/3306

No /admin alert, logging of long queries not included


(Kane York) #5