Skip Menu |
 

This queue is for tickets about the Apache-DBI CPAN distribution.

Report information
The Basics
Id: 28824
Status: resolved
Priority: 0/
Queue: Apache-DBI

People
Owner: pgollucci [...] p6m7g8.com
Requestors: perrin [...] elem.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: doc patch
Download (untitled) / with headers
text/plain 137b
I thought it would be nice to bring these docs up-to-date with modern DBI practices. I also corrected some grammar while I was in there.
Subject: apache-dbi.pm.patch
Download apache-dbi.pm.patch
text/x-diff 7.5k
--- lib/Apache/DBI.pm 2007-03-23 04:56:42.000000000 -0400 +++ lib/Apache/DBI.pm.perrin 2007-08-13 20:20:01.000000000 -0400 @@ -373,9 +373,9 @@ http://dbi.perl.org/ When loading the DBI module (do not confuse this with the Apache::DBI module) -it looks if the environment variable GATEWAY_INTERFACE starts with 'CGI-Perl' +it checks if the environment variable 'MOD_PERL' has been set and if the module Apache::DBI has been loaded. In this case every connect -request will be forwarded to the Apache::DBI module. This looks if a database +request will be forwarded to the Apache::DBI module. This checks if a database handle from a previous connect request is already stored and if this handle is still valid using the ping method. If these two conditions are fulfilled it just returns the database handle. The parameters defining the connection have @@ -386,14 +386,18 @@ because the Apache::DBI module overloads the disconnect method. The Apache::DBI module still has a limitation: it keeps database connections -persistent on a per process basis. The problem is, if a user accesses several -times a database, the http requests will be handled very likely by different -servers. Every server needs to do its own connect. It would be nice, if all -servers could share the database handles. Currently this is not possible, -because of the distinct name-space of every process. Also it is not possible -to create a database handle upon startup of the httpd and then inheriting this +persistent on a per process basis. The problem is, if a user accesses a database +several times, the http requests will be handled very likely by different +processes. Every process needs to do its own connect. It would be nice if all +servers could share the database handles, but currently this is not possible +because of the distinct memory-space of each process. Also it is not possible +to create a database handle upon startup of the httpd and then inherit this handle to every subsequent server. This will cause clashes when the handle is -used by two processes at the same time. +used by two processes at the same time. Apache::DBI has built-in protection +against this. It will not make a connection persistent if it sees that it is +being opened during the server startup. This allows you to safely open a connection +for grabbing data needed at startup and disconnect it normally before the end of +startup. With this limitation in mind, there are scenarios, where the usage of Apache::DBI is depreciated. Think about a heavy loaded Web-site where every @@ -402,15 +406,15 @@ time this would kill the web server. Another problem are timeouts: some databases disconnect the client after a -certain time of inactivity. The module tries to validate the database handle -using the ping-method of the DBI-module. This method returns true as default. -If the database handle is not valid and the driver has no implementation for -the ping method, you will get an error when accessing the database. As a -work-around you can try to replace the ping method by any database command, -which is cheap and safe or you can deactivate the usage of the ping method -(see CONFIGURATION below). +certain period of inactivity. The module tries to validate the database handle +using the C<ping()> method of the DBI-module. This method returns true by default. +Most DBI drivers have a working C<ping()> method, but if the driver you're using +doesn't have one and the database handle is no longer valid, you will get an error +when accessing the database. As a work-around you can try to add your own C<ping()> +method using any database command which is cheap and safe, or you can deactivate the +usage of the ping method (see CONFIGURATION below). -Here is generalized ping method, which can be added to the driver module: +Here is a generalized ping method, which can be added to the driver module: package DBD::xxx::db; # ====== DATABASE ====== use strict; @@ -430,12 +434,12 @@ Transactions: a standard DBI script will automatically perform a rollback whenever the script exits. In the case of persistent database connections, the database handle will not be destroyed and hence no automatic rollback -occurs. At a first glance it seems even to be possible, to handle a transaction +will occur. At a first glance it even seems possible to handle a transaction over multiple requests. But this should be avoided, because different -requests are handled by different servers and a server does not know the state -of a specific transaction which has been started by another server. In general +requests are handled by different processes and a process does not know the state +of a specific transaction which has been started by another process. In general, it is good practice to perform an explicit commit or rollback at the end of -every script. In order to avoid inconsistencies in the database in case +every request. In order to avoid inconsistencies in the database in case AutoCommit is off and the script finishes without an explicit rollback, the Apache::DBI module uses a PerlCleanupHandler to issue a rollback at the end of every request. Note, that this CleanupHandler will only be used, if @@ -447,8 +451,8 @@ This module plugs in a menu item for Apache::Status or Apache2::Status. The menu lists the current database connections. It should be considered incomplete because of the limitations explained above. It shows the current -database connections for one specific server, the one which happens to serve -the current request. Other servers might have other database connections. +database connections for one specific process, the one which happens to serve +the current request. Other processes might have other database connections. The Apache::Status/Apache2::Status module has to be loaded before the Apache::DBI module ! @@ -461,7 +465,7 @@ It is important, to load this module before any other modules using DBI ! -A common usage is to load the module in a startup file via the PerlRequire +A common usage is to load the module in a startup file called via the PerlRequire directive. See eg/startup.pl and eg/startup2.pl for examples. There are two configurations which are server-specific and which can be done @@ -494,7 +498,7 @@ =head2 MOD_PERL 2.0 -Apache::DBI version 0.96 and should work under mod_perl 2.0 RC5 and later +Apache::DBI version 0.96 and later should work under mod_perl 2.0 RC5 and later with httpd 2.0.49 and later. Apache::DBI versions less than 1.00 are NO longer supported. Additionally, @@ -503,12 +507,21 @@ =head2 MOD_PERL 1.0 Note that this module needs mod_perl-1.08 or higher, apache_1.3.0 or higher and that mod_perl needs to be configured with the appropriate call-back hooks: + + PERL_CHILD_INIT=1 PERL_STACKED_HANDLERS=1 Apache::DBI v0.94 was the last version before dual mod_perl 2.x support was begun. It still recommened that you use the latest version of Apache::DBI because Apache::DBI versions less than 1.00 are NO longer supported. - PERL_CHILD_INIT=1 PERL_STACKED_HANDLERS=1. +=head1 DO YOU NEED THIS MODULE? + +Note that this module is intended for use in porting existing DBI code to mod_perl, +or writing code that can run under both mod_perl and CGI. If you are using a +database abstraction layer such as Class::DBI, Rose::DB::Object, or DBIx::Class that already manages persistent connections for you, there is no need to use this module +in addition. Similarly, if you are developing new code that is strictly for use in +mod_perl, you may choose to use C<< DBI->connect_cached() >> instead, but consider +adding an automatic rollback after each request, as described above. =head1 SEE ALSO
From: perrin [...] elem.com
Download (untitled) / with headers
text/plain 108b
I made a mistake about how Rose::DB::Object interacts with Apache::DBI. Here's a corrected patch. - Perrin
Download apache-dbi.pm.patch
text/x-diff 7.6k
--- lib/Apache/DBI.pm 2007-03-23 04:56:42.000000000 -0400 +++ lib/Apache/DBI.pm.perrin 2007-08-14 17:51:25.000000000 -0400 @@ -373,9 +373,9 @@ http://dbi.perl.org/ When loading the DBI module (do not confuse this with the Apache::DBI module) -it looks if the environment variable GATEWAY_INTERFACE starts with 'CGI-Perl' +it checks if the environment variable 'MOD_PERL' has been set and if the module Apache::DBI has been loaded. In this case every connect -request will be forwarded to the Apache::DBI module. This looks if a database +request will be forwarded to the Apache::DBI module. This checks if a database handle from a previous connect request is already stored and if this handle is still valid using the ping method. If these two conditions are fulfilled it just returns the database handle. The parameters defining the connection have @@ -386,14 +386,18 @@ because the Apache::DBI module overloads the disconnect method. The Apache::DBI module still has a limitation: it keeps database connections -persistent on a per process basis. The problem is, if a user accesses several -times a database, the http requests will be handled very likely by different -servers. Every server needs to do its own connect. It would be nice, if all -servers could share the database handles. Currently this is not possible, -because of the distinct name-space of every process. Also it is not possible -to create a database handle upon startup of the httpd and then inheriting this +persistent on a per process basis. The problem is, if a user accesses a database +several times, the http requests will be handled very likely by different +processes. Every process needs to do its own connect. It would be nice if all +servers could share the database handles, but currently this is not possible +because of the distinct memory-space of each process. Also it is not possible +to create a database handle upon startup of the httpd and then inherit this handle to every subsequent server. This will cause clashes when the handle is -used by two processes at the same time. +used by two processes at the same time. Apache::DBI has built-in protection +against this. It will not make a connection persistent if it sees that it is +being opened during the server startup. This allows you to safely open a connection +for grabbing data needed at startup and disconnect it normally before the end of +startup. With this limitation in mind, there are scenarios, where the usage of Apache::DBI is depreciated. Think about a heavy loaded Web-site where every @@ -402,15 +406,15 @@ time this would kill the web server. Another problem are timeouts: some databases disconnect the client after a -certain time of inactivity. The module tries to validate the database handle -using the ping-method of the DBI-module. This method returns true as default. -If the database handle is not valid and the driver has no implementation for -the ping method, you will get an error when accessing the database. As a -work-around you can try to replace the ping method by any database command, -which is cheap and safe or you can deactivate the usage of the ping method -(see CONFIGURATION below). +certain period of inactivity. The module tries to validate the database handle +using the C<ping()> method of the DBI-module. This method returns true by default. +Most DBI drivers have a working C<ping()> method, but if the driver you're using +doesn't have one and the database handle is no longer valid, you will get an error +when accessing the database. As a work-around you can try to add your own C<ping()> +method using any database command which is cheap and safe, or you can deactivate the +usage of the ping method (see CONFIGURATION below). -Here is generalized ping method, which can be added to the driver module: +Here is a generalized ping method, which can be added to the driver module: package DBD::xxx::db; # ====== DATABASE ====== use strict; @@ -430,12 +434,12 @@ Transactions: a standard DBI script will automatically perform a rollback whenever the script exits. In the case of persistent database connections, the database handle will not be destroyed and hence no automatic rollback -occurs. At a first glance it seems even to be possible, to handle a transaction +will occur. At a first glance it even seems possible to handle a transaction over multiple requests. But this should be avoided, because different -requests are handled by different servers and a server does not know the state -of a specific transaction which has been started by another server. In general +requests are handled by different processes and a process does not know the state +of a specific transaction which has been started by another process. In general, it is good practice to perform an explicit commit or rollback at the end of -every script. In order to avoid inconsistencies in the database in case +every request. In order to avoid inconsistencies in the database in case AutoCommit is off and the script finishes without an explicit rollback, the Apache::DBI module uses a PerlCleanupHandler to issue a rollback at the end of every request. Note, that this CleanupHandler will only be used, if @@ -447,8 +451,8 @@ This module plugs in a menu item for Apache::Status or Apache2::Status. The menu lists the current database connections. It should be considered incomplete because of the limitations explained above. It shows the current -database connections for one specific server, the one which happens to serve -the current request. Other servers might have other database connections. +database connections for one specific process, the one which happens to serve +the current request. Other processes might have other database connections. The Apache::Status/Apache2::Status module has to be loaded before the Apache::DBI module ! @@ -461,7 +465,7 @@ It is important, to load this module before any other modules using DBI ! -A common usage is to load the module in a startup file via the PerlRequire +A common usage is to load the module in a startup file called via the PerlRequire directive. See eg/startup.pl and eg/startup2.pl for examples. There are two configurations which are server-specific and which can be done @@ -494,7 +498,7 @@ =head2 MOD_PERL 2.0 -Apache::DBI version 0.96 and should work under mod_perl 2.0 RC5 and later +Apache::DBI version 0.96 and later should work under mod_perl 2.0 RC5 and later with httpd 2.0.49 and later. Apache::DBI versions less than 1.00 are NO longer supported. Additionally, @@ -503,12 +507,23 @@ =head2 MOD_PERL 1.0 Note that this module needs mod_perl-1.08 or higher, apache_1.3.0 or higher and that mod_perl needs to be configured with the appropriate call-back hooks: + + PERL_CHILD_INIT=1 PERL_STACKED_HANDLERS=1 Apache::DBI v0.94 was the last version before dual mod_perl 2.x support was begun. It still recommened that you use the latest version of Apache::DBI because Apache::DBI versions less than 1.00 are NO longer supported. - PERL_CHILD_INIT=1 PERL_STACKED_HANDLERS=1. +=head1 DO YOU NEED THIS MODULE? + +Note that this module is intended for use in porting existing DBI code to mod_perl, +or writing code that can run under both mod_perl and CGI. If you are using a +database abstraction layer such as Class::DBI or DBIx::Class that already manages persistent connections for you, there is no need to use this module +in addition. (Another popular choice, Rose::DB::Object, can cooperate with +Apache::DBI or use your own custom connection handling.) If you are developing +new code that is strictly for use in mod_perl, you may choose to use +C<< DBI->connect_cached() >> instead, but consider adding an automatic rollback +after each request, as described above. =head1 SEE ALSO
Subject: Re: [rt.cpan.org #28824] doc patch
Date: Tue, 14 Aug 2007 23:38:21 -0400 (EDT)
To: "perrin [...] elem.com via RT" <bug-Apache-DBI [...] rt.cpan.org>
From: "Philip M. Gollucci" <pgollucci [...] p6m7g8.com>
Download (untitled) / with headers
text/plain 739b
Hey, bit busy at the moment, I'll to get this early next week. If you have karma go head and commit, I trust you. On Tue, 14 Aug 2007, perrin@elem.com via RT wrote: Show quoted text
> > Queue: Apache-DBI > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=28824 > > > I made a mistake about how Rose::DB::Object interacts with Apache::DBI. > Here's a corrected patch. > > - Perrin >
-- ------------------------------------------------------------------------ Philip M. Gollucci (philip@riderway.com) 323.219.4708 Senior System Admin - Riderway, Inc. http://riderway.com 1024D/EC88A0BF 0DE5 C55C 6BF3 B235 2DAB B89E 1324 9B4F EC88 A0BF Work like you don't need the money, love like you'll never get hurt, and dance like nobody's watching.
Part of 1.07 now on CPAN


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.