On Dienstag, 22. Mai 2007, you wrote: > > during my work with the database schemas i've noticed and fixed some > > issues with the current schema: > > > > - addition of a version field to pdt table > > Then the pdt module should also check the version number.
Hello Klaus, ok, that make sense. I'll look into this next week. > > - change uri: uri_user field length from 50 to 64 (standard user data > > length) > > Is this a global #define or is it hardcoded in every table definition? Of course a #define. :-) All these values are in db/schema/entities.xml defined. > > - change sip_trace: callid and :status from 254 to 255, and sip_trace: > > [..] > > - presentity: domain from 124 to 128 increased > > There is also mismatch between column types, maybe you can take a look > at this too. > Postgresql: > presentity.body is a "bytea" > xcap_xml.xcap is a "text" This is already fixed in the new schema. > > - xcap_xml: user from 66 to 64 decreased > > - pua:watcher_uri and pres_user increased from 128 to 255 (std. URI > > length) > > > > In my opinion it's more resonable to have some standard length for > > datatypes, what do you think about this changes? > > sounds reasonable > > > At the moment the IDs for the tables are either signed or unsigned. > > Should i change this to only one type? > > I think unsigned sounds better for an ID, but AFAIK postgresql does not > support unsigned integer. The "serial" is some kind of unsigned integer. Ok, good. Postgresql uses already "INTEGER(10) SERIAL NOT NULL" for the id fields. Henning _______________________________________________ Devel mailing list Devel@openser.org http://openser.org/cgi-bin/mailman/listinfo/devel