Hi Alex, Understood - as is where is. I happy to see postgres support, thanks for your effort here. Will do some stress testing and provide feedback.
Cheers, Alan On Sun, 2004-08-08 at 06:58, Alexander Malysh wrote: > Hi, > > please note, that dlr support for postgresql was commited by me and as > stated on the ML was not stress tested! > > > Alan McNatty wrote: > > > I agree completely - just highlighting for Bruno. > > > > If he's going to make a Debian package for stable it will be an issue if > > package has is compiled --with-pgsql and as long as 7.2 is in stable. > > > > On Fri, 2004-08-06 at 10:04, Pedro B. wrote: > >> Sorry, but doing patches to support a 7.2.x version when 7.4.x is the > >> stable version, is like proposing to go back to stoneage and make fire > >> by using 2 sticks. > >> > >> The 7.2 is completely deprecated on a postgres point of view. > >> > >> I do know debian-stable still uses it, but it shouldn't. 7.4.3 should > >> have gone out of testing ages ago. 7.5 is halfway under development, > >> because everyone in the "postgres world" (hate these tags, but they are > >> sometimes needed) has implemented 7.4. > >> > >> These are the 2 cents from a deep-into-postgres-point of view. > >> > >> \\pb > >> > >> > >> > >> > >> Alan McNatty wrote: > >> > >> > Hi Bruno, > >> > > >> > I've tested the pgsql dlr storage on 7.2.1 (debian stable) but found > >> > that Kannel utilised a DELETE ... LIMIT 1 in dlr_pgsql_remove from > >> > gw/dlr_pgsql.c which is not supported in 7.2.1. > >> > > >> > ERROR: PGSQL: DELETE FROM dlr WHERE smsc='foo' AND ts='243251116' LIMIT > >> > 1; > >> > ERROR: PGSQL: ERROR: parser: parse error at or near "LIMIT" > >> > ERROR: PGSQL: DB update failed! > >> > > >> > For postgres stable you may want to change from ... > >> > > >> > DELETE FROM a WHERE b=3 LIMIT 1; > >> > > >> > - to - > >> > > >> > DELETE FROM a WHERE ctid=(SELECT ctid FROM a WHERE b=3 LIMIT 1); > >> > > >> > Can supply a patch if required. > >> > > >> > Cheers, > >> > Alan > >> > > >> > > >> > On Fri, 2004-08-06 at 01:09, Bruno Rodrigues wrote: > >> > > >> >>Alexander Malysh wrote: > >> >> > >> >> > >> >>>hmm, ok - understand... looking through bug reports, I don't see any > >> >>>showstopper to call 1.3.2 -> 1.2.2 (anyway 1.3.2 is more stable as > >> >>>1.2.1), so I'm +1. > >> >> > >> >>Everybody else, "please vote now or shutup forever" ;) (how is it the > >> >>setence when you marry, in english ?) > >> >> > >> >> > >> >>>btw. copyright in debian directory doesn't match kannel license... > >> >> > >> >>Indeed. I haven't had time to update any debian stuff for too much > >> >>time. > >> >> I'll check everything and post it on my site and kannel before I try > >> >>to upload to Debian. Hopefully, still this evening. > >> >> > >> >> > >> >> > >> > > >> > > >> > > >> > > >> > >>
