On Thu, 29 May 2003, Tim Bunce wrote:
> > List of Changes:
> > 
> >         * Updated type_info_all() to be more inline with 
> >           what DBD::ODBC returns.
> >         * Added attribute 'auto_reconnect' that allows the auto reconnect
> >           behaviour to be toggled.
> 
> That should be 'mysql_auto_reconnect' of course.

Yes.

> 
> >           Right now this attribute defaults to 
> >           ON for backward compatibility; however, the default behaviour 
> >           may change so that auto_reconnect only defaults to on 
> >           in a mod_perl env.
> 
> I hope it will before the release. The current behaviour is unsafe for
> any application using locks.
> 

OK. I agree with you. -- The reason why I left the behaviour the way 
it was is because do do that you have override the mysql library connect 
defaults.

> >         * Added statistics attribute, 'mysql_dbd_stats' which returns
> >           a hashref that contains 2 keys 'auto_reconnects' and
> >           'failed_auto_reconnects'.
> 
> I'm a bit of a pedant when it comes to lexical ordering of things.
> I'd prefer 'auto_reconnects' and 'auto_reconnects_failed' (assuming
> auto_reconnects counts attempts, not just successes, otherwise it
> should be something like auto_reconnects_okay).
> 

OK.... It did not count attempts but rather sucesses. That is why I put 
the failed_, because I thought that made the behavour a little more 
obvious.  I will change them to 'auto_reconnects_okay' and 
'auto_reconnect_failed'.

> >         * Fixed bug where strings that were used in numeric
> >           context were not getting quoted on execute(). Now all
> >           parameters are bound as varchar by default.
> > 
> >           **NOTE** this is a chage in behaviour that MAY cause problems
> >           with some SQL statements. If quoted integers causes any 
> 
> s/integers/integers, for example,/
> 

OK IC, other numberics might cause problems also. will update.


Rudy.

Reply via email to