experimental support for Pg loader run inside txn
authorRafael Kitover <rkitover@cpan.org>
Tue, 7 Feb 2012 00:31:42 +0000 (19:31 -0500)
committerRafael Kitover <rkitover@cpan.org>
Tue, 7 Feb 2012 00:45:17 +0000 (19:45 -0500)
commit073070141b9041dbac8e375e36572b03755291cb
tree330db7f654bc354bdcf0d531325787e00be65a0d
parent9d78b02b432c530131dd5bbc56892d1203a9b5eb
experimental support for Pg loader run inside txn

To get this to work, there must not be any database errors thrown. So we
cannot use our usual technique of selecting from nonexistant tables and
throwing away the errors.

To that end, _table_comment and _column_comment in ::Loader::DBI now
check for the tables in $self->_tables before doing a query on them.

jnap added a test to the common tests in a previous commit which I
augment here to be $TODO for DBs other than Pg and to throw away the
usual rescan warnings.

There is also a warning for 'commit ineffective with AutoCommit enabled'
which we also throw away and is more worrisome, and will need further
attention, and is one of the reason this support is marked experimental.
That and this test failing for SQLite as well as some other DBs.
Changes
lib/DBIx/Class/Schema/Loader.pm
lib/DBIx/Class/Schema/Loader/DBI.pm
t/lib/dbixcsl_common_tests.pm