Hi Daniel,

rls_watchers is the same as active_watchers (holding state for presence dialogs).  No persistent data is held in these tables.

Hugh (who did the extra headers development) is looking at the silo table.

Regards,

Peter

On Fri, 2012-05-04 at 09:40 +0200, Daniel-Constantin Mierla wrote: 
Hello,

looking at changes in the database structure from 3.2 to 3.3, I noticed 
the columns in active_watchers/rls_watches that have no default value 
and cannot be NULL.

The usual issue with this kind of upgrades is that existing rows have to 
be removed. active_watchers is with temporary records, and I guess that 
is the same with rls_watchers. No persistent data is to be lost, right?

Another one is extra_hdrs in silo table. I think it should be allowed to 
be NULL or default '' (empty string), so stored messages will not be 
lost - the check over the code showed inserting '' if no extra headers 
are present at message storage time.

Cheers,
Daniel



-- 
Peter Dunkley
Technical Director
Crocodile RCS Ltd