Merge 'trunk' into 'replication_dedux'

r13471@dev (orig r4428):  matthewt | 2008-05-30 07:36:19 -0500
fix SQL::Translator parser to add $table_idx_ on the front of index names to prevent clashes
r13472@dev (orig r4429):  castaway | 2008-05-30 08:20:26 -0500
Started doc standards doc in Manual::Reading.
Fixed up belongs_to example to put more emphasis on the "column" part of "foreign_key_column" and corrected first paragraph.

r13473@dev (orig r4430):  ash | 2008-05-30 08:29:32 -0500
Make ingore_{constraint,index}_names default
r13474@dev (orig r4431):  ash | 2008-05-30 08:34:39 -0500
Update docs (and code changes that were missed from last commit, oops):
Set ignore_contraint_names to default on

Update docs (and code changes that were missed from last commit, oops):
Set ignore_contraint_names to default on

Make ingore_{constraint,index}_names default

Started doc standards doc in Manual::Reading.
Fixed up belongs_to example to put more emphasis on the "column" part of "foreign_key_column" and corrected first paragraph.

fix SQL::Translator parser to add $table_idx_ on the front of index names to prevent clashes

fixed failing test in podcoverage, fixed regression in replication test that caused the default fake sqlite replication to fail.  not so important since sqlite doesnt replicate but we do not like to see failing tests.

changed Storage->reload_row to do less, reverted some behavior to PK->discard_changes.  Did this to solve some capatibility issues with partitioning.  updated docs to reflect this.

discard changes now is forced to use master for replication.  changed discard_changes guts to point to new method called reload_row in storage.  fixed problem with replicated transactionws not returning the right thing.  added tests to all the above

created storage method to execute a coderef using master storage only, changed tnx_do to only use the master, wrote tests for both the above, wrote docs for both the above

Merge 'trunk' into 'replication_dedux'

r13166@dev (orig r4392):  castaway | 2008-05-19 16:57:25 -0500
Added doc for "for => update" attribute, thanks StuartL.

r13167@dev (orig r4393):  castaway | 2008-05-19 16:59:16 -0500
Oops, =cut after the pod not in the middle

r13272@dev (orig r4419):  ash | 2008-05-27 05:41:44 -0500
Update FAQ to mention behavour of scalar refs w.r.t. update
r13273@dev (orig r4420):  ash | 2008-05-27 07:49:09 -0500
Add set_cache example to cookbook
r13274@dev (orig r4421):  ash | 2008-05-27 07:50:10 -0500
Fix var name typo in cookbook
r13376@dev (orig r4422):  matthewt | 2008-05-27 08:31:58 -0500
version bump, deprecated ResultSetManager

version bump, deprecated ResultSetManager

Fix var name typo in cookbook

Add set_cache example to cookbook

Update FAQ to mention behavour of scalar refs w.r.t. update

Oops, =cut after the pod not in the middle

Added doc for "for => update" attribute, thanks StuartL.

make sure that the Pool validates the replicants on the first query

documentation updates

changed SQLT::Parser::DBIC so that in FK constraints, both the field order for the source and target tables are ensured to follow the same ordering rule

Merge 'trunk' into 'replication_dedux'

r12028@dev (orig r4378):  captainL | 2008-05-11 13:43:58 -0500
added test for find or create related functionality in nested inserts
r12029@dev (orig r4379):  captainL | 2008-05-11 14:03:27 -0500
sanified new multi_create test
r12030@dev (orig r4381):  ribasushi | 2008-05-12 06:09:20 -0500
Add failing tests for missing safeguards for multilevel prefetch (don't know the internals well enough to fix the issue itself)

Add failing tests for missing safeguards for multilevel prefetch (don't know the internals well enough to fix the issue itself)

sanified new multi_create test

added test for find or create related functionality in nested inserts

removed bad tabbing

doh, replaced ability to configure pool args via the storage args

changed the way args are passed to a storage type that needs args so they can be in the form of a hash or array ref.  This plays nicer with Config::General for loading

changed the balancer to a role, created a new class to define the default balancer behavior, finished the autovalidate code and added tests for all the above

Merge 'trunk' into 'replication_dedux'

r8571@dev (orig r4363):  bricas | 2008-05-08 10:44:24 -0500
add Test::Exception to deps (RT #34256). move testing modules to test_requires().
r8572@dev (orig r4364):  bricas | 2008-05-08 15:08:18 -0500
add RT ticket number to changes
r8579@dev (orig r4371):  bricas | 2008-05-08 21:55:30 -0500
pod fix (RT #32988)

pod fix (RT #32988)

good start on the validation of replicants and a system to automatically validate them (although that might be a better role than inside a class, for someday!)

fixed up the relicant status tests to exclude them if the database is not a real replicating setup, removed some debug code, got the lag_behind_master and is_replicating methods working properly.

fixed up the relicant status tests to exclude them if the database is not a real replicating setup, removed some debug code, got the lag_behind_master and is_replicating methods working properly.

converted replicant to a role so that we can apply it after ensure_connected properly reblesses the storage into the correct driver specific storage

more cleanup of the test suite so that we can run it against other databases.  fixed the problem with tests using self-referential constrainsts for dbs that have trouble handling that

lots of updates to make the test suite work with databases other than sqlite

add RT ticket number to changes

add Test::Exception to deps (RT #34256). move testing modules to test_requires().

Merge 'trunk' into 'replication_dedux'

r8568@dev (orig r4360):  nigel | 2008-05-08 08:21:33 -0500
cascade_copy docs - takes from http://osdir.com/ml/lang.perl.modules.dbix-class/2007-03/msg00106.html
r8569@dev (orig r4361):  nigel | 2008-05-08 08:22:44 -0500
cascade_copy docs (format fixed)

cascade_copy docs (format fixed)

cascade_copy docs - takes from osdir.com/ml/lang.perl.modules.dbix-class/2007-03/msg00106.html

changed the way args are passed to a storage, should make it easier to use existing code using this, added the master as a fallback to the the replicants, lots of small documentation updates and test improvements.  all tests passing

Merge 'trunk' into 'replication_dedux'

r8564@dev (orig r4356):  ribasushi | 2008-05-07 10:25:41 -0500
Correct order of BEGIN/use so that skip_all does not produce failures
r8565@dev (orig r4357):  ribasushi | 2008-05-07 11:37:56 -0500
Various fixes to make testing with DBICTEST_SQLT_DEPLOY=1 possible

Various fixes to make testing with DBICTEST_SQLT_DEPLOY=1 possible

Correct order of BEGIN/use so that skip_all does not produce failures

Merge 'trunk' into 'replication_dedux'

r8495@dev (orig r4354):  ribasushi | 2008-05-07 08:33:50 -0500
Fix a forgotten CDBICompat component relocation

Fix a forgotten CDBICompat component relocation

documented methods for detecting replicant reliability, created stub methods

refactored the duties of the different balancer classes, added tests and docs

Merge 'trunk' into 'replication_dedux'

updated Changes to document Replicated Storage type changes

changed the warning on DBIC::Storage::DBI->select_single so that it wont call fetch_* on an empty sth, updated tests for the above and added a todo test for the wrong count problem

Note that single expects a single row in the ResultSet docs

removed code that tossed an error in select_single when more than a single row is returned and updated the tests to TODO the bad count issue

Merge 'trunk' into 'replication_dedux'

r8434@dev (orig r4321):  matthewt | 2008-05-05 12:55:23 -0500
reverse r4290 since we -do- -not- currently want these namespaces indexed
r8435@dev (orig r4326):  jnapiorkowski | 2008-05-05 13:33:57 -0500
removed query count stuff from trunk so we can play with this on a branch instead
r8436@dev (orig r4330):  matthewt | 2008-05-05 14:59:31 -0500
this was meant to be deleted with the rest of the Moose stuff
r8437@dev (orig r4331):  matthewt | 2008-05-05 15:00:18 -0500
make override follow Replication -> Replicated rename
r8438@dev (orig r4332):  matthewt | 2008-05-05 15:00:49 -0500
restore debolaz's cleanup patch since it's not what I thought it was and probably the right thing to do; mea culpa
r8439@dev (orig r4333):  matthewt | 2008-05-05 15:22:17 -0500
missed another package line
r8440@dev (orig r4334):  matthewt | 2008-05-05 15:32:58 -0500
improvements to the META.yml magic
r8442@dev (orig r4336):  matthewt | 2008-05-05 15:46:21 -0500
no author notes in Changes if you're already in CONTRIBUTORS please, once you have that and a commit bit you're just another poor bastard stuck maintaining an ORM, just like I am

no author notes in Changes if you're already in CONTRIBUTORS please, once you have that and a commit bit you're just another poor bastard stuck maintaining an ORM, just like I am

updated mysql test to reflect the fetch without execute error

improvements to the META.yml magic

missed another package line

restore debolaz's cleanup patch since it's not what I thought it was and probably the right thing to do; mea culpa

make override follow Replication -> Replicated rename

this was meant to be deleted with the rest of the Moose stuff

removed query count stuff from trunk so we can play with this on a branch instead

reverse r4290 since we -do- -not- currently want these namespaces indexed

changed replication test to support custom database connect info, added a little code to DBICTest to support this

added test to check when nothing is found

cleanup of some docs, got the default shuffling balancer to work properly.  Don't fall in love with this behavior, since I'm probably going to change the default to balancer to just return the first replicant in the list, since this is optimised for the common case of a single replicant

added some advice to debugging replicants so that we can see a replicant dsn, got this balancing between one master and slave, seems to run well

added some advice to debugging replicants so that we can see a replicant dsn, got this balancing between one master and slave, seems to run well

got first pass on the replication and balancer, passing all of the old test suite (which is not much, but it is a milestone of some sort)

new config option to DBICTest to let you set an alternative storage type, start on creating a DBIC based load balancer

minor formatting updates and typos fixes to the schema role tests

fixed up the replication test, added some tests for the dbd::multi problem of null results, fixed replicated.pm docs and a few typos

renamed replication storage to replicated (since it's not really doing any replication itself)

removed auto-generated indexes from being explicitly created

Fix PAUSE indexing of packages

todo test from zby for set_from_related

added role for the at query interval run job system

Add a proxy deploy_statements() method executable directly on $schema instead of $schema->storage

TODO tests for multiple same level has_many's
Currenty both exception checks pass - should fail until the code is in place

Split prefetch related tests from 76joins.t

add TODO on constraint check

fix loading checks

querycounter role, test for that and a new schema hierarchy for additional Moose related development

removed pointless debugging module

- added tests to replication to make sure dbd::multi fails over when a dtabase is forcible disconnected
- updated DBIC::DBI::Replication so that in addition to a dsn string, you can pass a preexisting database handle.

Documentation updates from omega, carbon

Add warnings for non-unique find usage and improve explicit key attr handling in find (zby)

Fix a typo.

As of revision 4232, DBIx::Class uses Path::Class.  Add this module to
DBIx::Class's dependencies.

Missing file

Constraint/index name fix from rdj

Removing since Build.PL is not longer supported in modern M::I

fixed typos

added search_related_rs at resultset

fix pod fail

CDBICompat::Relationships uses Clone.  Make CDBICompat not load without it
so the tests skip if it's not there.

Also improve the message to only show what's missing.

Back all savepoint tests out of (mysql|pg).t

There's not a dire need to have them there, the only thing they
would be testing are the ->_svp_* methods on Storage, which are
tested in 98savepoints.t anyway.

I kept the changes in 72pg.t that was neccesary for it to run
to completion though.

And document the last changes

Various fun things.

DONE:
* We now maintain a stack for savepoints.
* Most savepoint tests are now in a separate test file.
* Arguments to svp_* methods are now optional.
* We throw exceptions instead of warn on many places.
* gphat++

TODO:
* Document all this.

Merge 'storage-ms-access' into 'trunk'