[DBIx-Class-Devel] [dbix-class] Disconnect after schema version check. (#57)

Alexander Hartmaier alex.hartmaier at gmail.com
Tue Aug 26 12:10:21 GMT 2014


Hi Eric,
did you test with the latest dev release of DBIC?

On Tue, Aug 26, 2014 at 6:37 AM, Peter Rabbitson <notifications at github.com>
wrote:

> First of all many thanks for the time looking into this! I would however
> like for you to look deeper into the issue. The current behavior seems
> wrong in itself, therefore the result is you adding a bandaid fixing
> something that should not have been there in the first place.
>
> The problem on the table is: we are connecting twice.
> The correct question isn't "why aren't we disconnecting", but rather "why
> are we connecting twice?!"
>
> The solution would lie in examining the ::Versioned code and determining
> whether any of the ->connect calls have anything special done to their
> connect attributes. If this is *not* the case (i.e. we always connect to
> the same place *anyway*) - then clearly you are looking into a shared
> $dbh as can be seen in the *second* bullet point here:
> https://metacpan.org/pod/DBIx::Class::Storage::DBI#connect_info. I will
> not be available for most of this week, please ask @frioux
> <https://github.com/frioux> for help if you get stuck and/or have trouble
> navigating my explanation.
>

@Peter: this sounds to me like a consequence of the connection race
condition we debugged and fixed some month ago where the wrong connection
got the session parameters set but the other one got used.


>> Reply to this email directly or view it on GitHub
> <https://github.com/dbsrgits/dbix-class/pull/57#issuecomment-53374573>.
>
> _______________________________________________
> DBIx-Class-Devel mailing list
> DBIx-Class-Devel at lists.scsys.co.uk
> http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/dbix-class-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scsys.co.uk/pipermail/dbix-class-devel/attachments/20140826/5bd9046a/attachment.htm>


More information about the DBIx-Class-Devel mailing list