Code review comment for lp:~linaro-infrastructure/launchpad/notify-workitems-changes

Revision history for this message
Guilherme Salgado (salgado) wrote :

<lifeless> you might want to figure out why it has that decorator
<lifeless> this smells to me of some bug patched over
<lifeless> if it was e.g. a truism that all event handlers must not permit implicit flushes, we'd have glued it into the event subscription code to avoid the repetition
<lifeless> salgado: I would look in annotate for the commit adding the decorator and see what it was part of; look at the MP too probably; and definitely try removing the decorator and seeing what happens

« Back to merge proposal