Jade's weblog Archive

The problem was that the code said:

db_foreach select_forums { select forum_id from ... } {
    notification::request::new ...
}

which caused the db_foreach and the notification::request::new to use different database handles, which happens to mean they'll not be part of the same transaction. Hence this caused a deadlock.

I serialized the two db-calls to say

foreach forum_id [db_list select_forums {}] {
    notification::request::new ...
}

so that the db_list call returns before the foreach loop starts, causing the two to use the same transaction (or, rather, the inner call uses the active transaction).