Bug #9039 for DBD-mysql: [REQ] ability to define reconnect sub used by 'auto

Mon Dec 20 07:24:47 2004 Guest - Ticket created

Subject:

[REQ] ability to define reconnect sub used by 'auto_reconnect'

Hiya, I'd like to be able to define the subroutine used when an auto reconnect occurs. The reason being for failover and redundancy: Say I have 3 mysql servers that are read only -- if my connection to one of them times out, instead of just reconnecting to it I'd like to be able to decide which one of the three I should reconnect to, taking things like server load, # connections, etc. into account. AFAIKS, this would involve making a small-ish change to: int mysql_db_reconnect(SV* h) { To allow an external perl sub to be called if set, instead of calling _MyLogin( imp_dbh ). This could be implemented by adding the reconnector sub as another 'attribute' of 'imp_dbh', with the default behaviour being to call a perl sub that just calls _MyLogin(). -Steve

Fri Mar 04 15:01:54 2005 rlippan [...] remotelinux.com - Correspondence added

[guest - Mon Dec 20 07:24:47 2004]: Show quoted text

> Hiya, >

Hello. Show quoted text

> I'd like to be able to define the subroutine used when an auto > reconnect occurs. The reason being for failover and redundancy: > Say I have 3 mysql servers that are read only -- if my connection > to one of them times out, instead of just reconnecting to it I'd > like to be able to decide which one of the three I should reconnect > to, taking things like server load, # connections, etc. into > account.

IMO, this would be better handled in an external module and not by the driver itself. As it stands right now auto_reconnect is more of a backwards compat. feature who's behaviour changes depending on context (mod_perl/!mod_perl, flag set/ in_begin? in_tx &c), so it really shouldn't be used. Show quoted text

> > AFAIKS, this would involve making a small-ish change to: > > int mysql_db_reconnect(SV* h) { > > To allow an external perl sub to be called if set, instead of calling > _MyLogin( imp_dbh ). This could be implemented by adding the > reconnector sub as another 'attribute' of 'imp_dbh', with the > default behaviour being to call a perl sub that just calls > _MyLogin().

Why bounce to perl and then back to C as a default when you could: If (imp_dbh->reconnect_callback()) return imp_dbh->reconnect_callback(imp_dbh);

Fri Mar 04 15:01:55 2005 rlippan [...] remotelinux.com - Status changed from 'new' to 'resolved'

Bug #9039 for DBD-mysql: [REQ] ability to define reconnect sub used by 'auto_reconnect'