Backout sybase changes

Clear up some legacy cruft and straighten inheritance

This code belogs in Storage::DBI

Changes

Nothing says the grouping column can not be nullable

Changes and dev notes in makefile

Minor cleanup of RSC with has_many joins

duh

Cleanup exception handling

Merge 'sybase' into 'trunk'

Rollback all bulk insert code before merge

Merge 'sybase_bulk_insert' into 'sybase'

r7678@Thesaurus (orig r7667):  ribasushi | 2009-09-16 19:31:14 +0200
New subbranch
r7679@Thesaurus (orig r7668):  ribasushi | 2009-09-16 19:34:29 +0200
Caelum's work so far
r7690@Thesaurus (orig r7679):  caelum | 2009-09-18 11:10:16 +0200
support for blobs in insert_bulk fallback

support for blobs in insert_bulk fallback

Merge 'sybase' into 'sybase_bulk_insert'

fix freetds

column no longer necessary in test

fix yesterday's stuff, identity_update works, blob updates are better

Fix left-join chaining

I'll rewrite this bit tomorrow to be less retarded

Merge 'sybase' into 'sybase_bulk_insert'

r7666@Thesaurus (orig r7655):  caelum | 2009-09-14 15:09:21 +0200
sybase bulk API support stuff (no blobs yet, coming soon...)
r7667@Thesaurus (orig r7656):  caelum | 2009-09-14 15:33:14 +0200
add another test for sybase bulk stuff (passes)
r7668@Thesaurus (orig r7657):  caelum | 2009-09-14 15:44:06 +0200
minor change (fix inverted boolean for warning)
r7669@Thesaurus (orig r7658):  caelum | 2009-09-14 15:48:52 +0200
remove @args from DBI::sth, use full arg list
r7676@Thesaurus (orig r7665):  caelum | 2009-09-16 15:06:35 +0200
use execute_array for insert_bulk, test insert_bulk with blobs, clean up blob tests a bit
r7677@Thesaurus (orig r7666):  caelum | 2009-09-16 15:09:23 +0200
 r20647@hlagh (orig r7659):  ribasushi | 2009-09-14 12:24:44 -0400
 Someone claimed this is a problem...
 r20650@hlagh (orig r7662):  ribasushi | 2009-09-15 03:43:46 -0400
 Warn when distinct is used with group_by
 r20651@hlagh (orig r7663):  rbuels | 2009-09-15 16:45:32 -0400
 doc patch, clarified warning about using find_or_create() and friends on tables with auto-increment or similar columns
 r20652@hlagh (orig r7664):  rbuels | 2009-09-15 16:55:15 -0400
 another doc clarification regarding auto-inc columns with find_or_create() and such functions

r7680@Thesaurus (orig r7669):  ribasushi | 2009-09-16 19:36:19 +0200
Remove branched changes

Remove branched changes

Caelum's work so far

Merge 'trunk' into 'sybase'

r20647@hlagh (orig r7659):  ribasushi | 2009-09-14 12:24:44 -0400
Someone claimed this is a problem...
r20650@hlagh (orig r7662):  ribasushi | 2009-09-15 03:43:46 -0400
Warn when distinct is used with group_by
r20651@hlagh (orig r7663):  rbuels | 2009-09-15 16:45:32 -0400
doc patch, clarified warning about using find_or_create() and friends on tables with auto-increment or similar columns
r20652@hlagh (orig r7664):  rbuels | 2009-09-15 16:55:15 -0400
another doc clarification regarding auto-inc columns with find_or_create() and such functions

use execute_array for insert_bulk, test insert_bulk with blobs, clean up blob tests a bit

another doc clarification regarding auto-inc columns with find_or_create() and such functions

doc patch, clarified warning about using find_or_create() and friends on tables with auto-increment or similar columns

Warn when distinct is used with group_by

Someone claimed this is a problem...

remove @args from DBI::sth, use full arg list

minor change (fix inverted boolean for warning)

add another test for sybase bulk stuff (passes)

sybase bulk API support stuff (no blobs yet, coming soon...)

allow subclassing of methods proxied to _writer_storage

make sure _init doesn't loop, steal insert_bulk from mssql, add some insert_bulk tests

Merge 'trunk' into 'sybase'

r20609@hlagh (orig r7621):  mo | 2009-09-10 12:53:32 -0400
added test to make sure that store_column is called even for non-dirty columns
r20610@hlagh (orig r7622):  bluefeet | 2009-09-10 13:03:21 -0400
Fix RSC->reset() to no longer return $self, which fixes Cursor::Cached + RSC.
r20611@hlagh (orig r7623):  ribasushi | 2009-09-10 13:32:03 -0400
The real fix
r20612@hlagh (orig r7624):  matthewt | 2009-09-10 20:33:17 -0400
make it clear that we are not supposed to have optional deps
r20613@hlagh (orig r7625):  ribasushi | 2009-09-11 00:30:03 -0400
Changes so far
r20614@hlagh (orig r7626):  ribasushi | 2009-09-11 00:39:45 -0400
Fix borked makefile
r20615@hlagh (orig r7627):  ribasushi | 2009-09-11 09:39:42 -0400
Fixed minor problem with txn scope guard - rollback exceptions were never reported
r20617@hlagh (orig r7629):  ribasushi | 2009-09-11 17:06:54 -0400
Extend prefetch tests
r20618@hlagh (orig r7630):  ribasushi | 2009-09-11 17:13:45 -0400
Reverting http://dev.catalyst.perl.org/svnweb/bast/revision?rev=4278 - it seems to pass fine now
r20619@hlagh (orig r7631):  ribasushi | 2009-09-11 18:15:50 -0400
Add single() ro RSC
r20620@hlagh (orig r7632):  ribasushi | 2009-09-11 18:44:01 -0400
This is how the txnguard should really work
r20621@hlagh (orig r7633):  ribasushi | 2009-09-11 18:58:21 -0400
Fix borked example
r20622@hlagh (orig r7634):  ribasushi | 2009-09-11 18:58:58 -0400
scopeguard almost done
r20623@hlagh (orig r7635):  brunov | 2009-09-11 19:25:12 -0400
Update DBIx::Class::Manual::Example.pod to reflect previous changes in examples/Schema/insertdb.pl

r20624@hlagh (orig r7636):  brunov | 2009-09-11 19:27:17 -0400
Added Bruno Vecchi to the Contributors section in DBIx/Class.pm

r20625@hlagh (orig r7637):  ribasushi | 2009-09-11 19:31:16 -0400
Final scopeguard tweak (?)
r20629@hlagh (orig r7641):  ribasushi | 2009-09-12 06:46:51 -0400
Even better localization of $@, and don't use Test::Warn for the time being, as something is freaking out Sub::UpLevel

change _insert_dbh to _insert_storage

Even better localization of $@, and don't use Test::Warn for the time being, as something is freaking out Sub::UpLevel

Final scopeguard tweak (?)

Added Bruno Vecchi to the Contributors section in DBIx/Class.pm

Update DBIx::Class::Manual::Example.pod to reflect previous changes in examples/Schema/insertdb.pl

scopeguard almost done

Fix borked example

This is how the txnguard should really work

Add single() ro RSC

Reverting dev.catalyst.perl.org/svnweb/bast/revision?rev=4278 - it seems to pass fine now

Extend prefetch tests

Fixed minor problem with txn scope guard - rollback exceptions were never reported

Fix borked makefile

Changes so far

make it clear that we are not supposed to have optional deps

The real fix

Fix RSC->reset() to no longer return $self, which fixes Cursor::Cached + RSC.

added test to make sure that store_column is called even for non-dirty columns

do blob update over _insert_dbh

remove Sub::Name hack for method dispatch, pass $next instead

Merge 'trunk' into 'sybase'

r20588@hlagh (orig r7613):  ribasushi | 2009-09-09 08:16:12 -0400
Fix warning
r20589@hlagh (orig r7614):  ribasushi | 2009-09-09 08:42:49 -0400
Really sanify exception text

Really sanify exception text

Fix warning

Merge 'trunk' into 'sybase'

r20554@hlagh (orig r7595):  wreis | 2009-09-07 09:31:38 -0400
improved warn for Storable hooks in ResultSourceHandle
r20558@hlagh (orig r7597):  ribasushi | 2009-09-07 10:26:59 -0400
Whoops - last_insert_id allows for multiple autoinc columns - support it in pg
r20560@hlagh (orig r7598):  ribasushi | 2009-09-07 10:46:14 -0400
Prune duplicate constraints from the find() condition
r20578@hlagh (orig r7603):  frew | 2009-09-08 14:13:29 -0400
Turn IDENTITY_INSERT back off after inserts

test multi-row blob update

This should not have worked - I guess we lack tests?

Reduce amount of perl-golf :)

Much simpler

Localisation ain't free, we don't do it unless we have to

remove unsafe_insert

Turn IDENTITY_INSERT back off after inserts

fix _insert_dbh code to only connect when needed, doc update

Prune duplicate constraints from the find() condition

Whoops - last_insert_id allows for multiple autoinc columns - support it in pg

improved warn for Storable hooks in ResultSourceHandle

Merge 'trunk' into 'sybase'

r7511@Thesaurus (orig r7508):  matthewt | 2009-09-03 20:12:53 +0200
get the COPYRIGHT in the right pless to not confuse META.yml generation
r7513@Thesaurus (orig r7510):  ribasushi | 2009-09-03 20:41:22 +0200

r7514@Thesaurus (orig r7511):  ribasushi | 2009-09-03 20:41:34 +0200
 r7472@Thesaurus (orig r7469):  norbi | 2009-09-01 21:43:08 +0200
  r7635@vger:  mendel | 2009-09-01 21:02:23 +0200
  Added pointer to 'SQL functions on the lhs' to the 'using stored procs' section.

r7515@Thesaurus (orig r7512):  ribasushi | 2009-09-03 20:41:44 +0200
 r7473@Thesaurus (orig r7470):  norbi | 2009-09-01 21:43:19 +0200
  r7636@vger:  mendel | 2009-09-01 21:09:43 +0200
  Mentions the possibiliby of creating indexes on SQL function return values.

r7516@Thesaurus (orig r7513):  ribasushi | 2009-09-03 20:41:52 +0200
 r7474@Thesaurus (orig r7471):  norbi | 2009-09-01 21:43:31 +0200
  r7637@vger:  mendel | 2009-09-01 21:19:14 +0200
  Rewrote 'SQL functions on the lhs' to use the new SQLA literal SQL + bind feature.

r7517@Thesaurus (orig r7514):  ribasushi | 2009-09-03 20:41:59 +0200
 r7475@Thesaurus (orig r7472):  norbi | 2009-09-01 21:43:42 +0200
  r7638@vger:  mendel | 2009-09-01 21:20:17 +0200
  Added a comment to the example code to stress that it does not work.

r7518@Thesaurus (orig r7515):  ribasushi | 2009-09-03 20:42:10 +0200
 r7476@Thesaurus (orig r7473):  norbi | 2009-09-01 21:43:54 +0200
  r7639@vger:  mendel | 2009-09-01 21:28:18 +0200
  Added pointer to DBIx::Class::DynamicSubclass.

r7519@Thesaurus (orig r7516):  ribasushi | 2009-09-03 20:42:15 +0200
 r7477@Thesaurus (orig r7474):  norbi | 2009-09-01 21:44:03 +0200
  r7640@vger:  mendel | 2009-09-01 21:30:13 +0200
  Replaced deprecated \'colname DESC' order_by syntax with { -desc => 'colname' } syntax.

r7520@Thesaurus (orig r7517):  ribasushi | 2009-09-03 20:42:22 +0200
 r7478@Thesaurus (orig r7475):  norbi | 2009-09-01 21:44:17 +0200
  r7641@vger:  mendel | 2009-09-01 21:32:48 +0200
  Rewrote 'SQL functions on the lhs' to use the new SQLA literal SQL + bind feature.

r7521@Thesaurus (orig r7518):  ribasushi | 2009-09-03 20:42:26 +0200
 r7479@Thesaurus (orig r7476):  norbi | 2009-09-01 21:44:28 +0200
  r7642@vger:  mendel | 2009-09-01 21:42:25 +0200
  Added many-to-many add_to_*() example to stress that it returns the related row and not the linking table row.

r7522@Thesaurus (orig r7519):  ribasushi | 2009-09-03 20:42:32 +0200
 r7480@Thesaurus (orig r7477):  norbi | 2009-09-01 22:14:25 +0200
  r7653@vger:  mendel | 2009-09-01 22:14:11 +0200
  Fixed wrong literal SQL + bind examples (missing operator and placeholders).

r7523@Thesaurus (orig r7520):  ribasushi | 2009-09-03 20:42:37 +0200
 r7481@Thesaurus (orig r7478):  norbi | 2009-09-01 22:30:48 +0200
  r7655@vger:  mendel | 2009-09-01 22:30:35 +0200
  Fixed the bind value column names in the SQL literal + bind examples.

r7524@Thesaurus (orig r7521):  ribasushi | 2009-09-03 20:42:45 +0200
 r7482@Thesaurus (orig r7479):  norbi | 2009-09-01 22:52:21 +0200
  r7657@vger:  mendel | 2009-09-01 22:52:09 +0200
  Further improvement in the bind value column names in the SQL literal + bind examples.

r7549@Thesaurus (orig r7546):  ribasushi | 2009-09-04 08:47:19 +0200
Stop connecting to determine dt-parser (test is in pg branch)
r7553@Thesaurus (orig r7550):  ribasushi | 2009-09-04 11:20:48 +0200
Require sqla with bool support
r7560@Thesaurus (orig r7557):  ribasushi | 2009-09-04 19:17:32 +0200
Dumper follies
r7561@Thesaurus (orig r7558):  ribasushi | 2009-09-04 19:27:50 +0200
Even better sqla
r7570@Thesaurus (orig r7567):  ribasushi | 2009-09-04 20:49:53 +0200
 r7459@Thesaurus (orig r7456):  rbuels | 2009-09-01 12:46:46 +0200
 making another pg_unqualified_schema branch, for real this time
 r7460@Thesaurus (orig r7457):  rbuels | 2009-09-01 12:51:31 +0200
 reworked tests for pg last_insert_id in presence of un-schema-qualified things. adds some todo tests, including a case for which is does not seem to be possible to correctly guess the sequence to use for the liid
 r7461@Thesaurus (orig r7458):  rbuels | 2009-09-01 12:54:34 +0200
 in Pg storage, added a warning for case when the nextval sequence is not schema qualified
 r7462@Thesaurus (orig r7459):  rbuels | 2009-09-01 13:01:31 +0200
 tweak to Pg test, warnings_like -> warnings_exist
 r7463@Thesaurus (orig r7460):  ribasushi | 2009-09-01 13:34:59 +0200
 Rewrap todo properly
 r7490@Thesaurus (orig r7487):  ribasushi | 2009-09-02 14:16:01 +0200
 Make pg sequence autodetect deterministic (or throw exceptions). Test needs adjusting
 r7491@Thesaurus (orig r7488):  rbuels | 2009-09-02 19:15:01 +0200
 some reorganization and cleanup of pg-specific tests
 r7492@Thesaurus (orig r7489):  rbuels | 2009-09-02 20:08:31 +0200
 more cleanup of 72pg.t
 r7495@Thesaurus (orig r7492):  rbuels | 2009-09-02 20:48:12 +0200
 more cleanup of pg tests, added cascade to drop function, cleaned up create and drop of schemas to use dbh_do
 r7496@Thesaurus (orig r7493):  rbuels | 2009-09-02 20:50:42 +0200
 oops, missed something screwed up by the pull
 r7525@Thesaurus (orig r7522):  rbuels | 2009-09-03 20:45:53 +0200
 added __END__ before pod in Pg storage
 r7526@Thesaurus (orig r7523):  rbuels | 2009-09-03 20:46:00 +0200
 renamed pg test schemas to be more organized
 r7531@Thesaurus (orig r7528):  rbuels | 2009-09-04 00:28:11 +0200
 more pg test cleanup
 r7532@Thesaurus (orig r7529):  rbuels | 2009-09-04 00:28:17 +0200
 more pg test cleanup
 r7533@Thesaurus (orig r7530):  rbuels | 2009-09-04 00:28:25 +0200
 starting work on extended set of Pg auto-pk tests
 r7534@Thesaurus (orig r7531):  rbuels | 2009-09-04 00:28:31 +0200
 more work on extended set of Pg auto-pk tests
 r7535@Thesaurus (orig r7532):  rbuels | 2009-09-04 00:28:39 +0200
 more work on pg tests
 r7536@Thesaurus (orig r7533):  rbuels | 2009-09-04 00:28:45 +0200
 more work on extended set of Pg auto-pk tests
 r7537@Thesaurus (orig r7534):  rbuels | 2009-09-04 00:28:50 +0200
 added .gitignore for users of git-svn
 r7538@Thesaurus (orig r7535):  rbuels | 2009-09-04 00:28:58 +0200
 more work on extended set of Pg auto-pk tests
 r7539@Thesaurus (orig r7536):  rbuels | 2009-09-04 00:29:04 +0200
 added darcs and git to MANIFEST.SKIP version control skipping section
 r7540@Thesaurus (orig r7537):  rbuels | 2009-09-04 00:41:26 +0200
 more work on extended set of Pg auto-pk tests
 r7541@Thesaurus (orig r7538):  rbuels | 2009-09-04 00:41:32 +0200
 more work on extended set of Pg auto-pk tests
 r7542@Thesaurus (orig r7539):  rbuels | 2009-09-04 00:41:38 +0200
 more work on extended set of Pg auto-pk tests
 r7543@Thesaurus (orig r7540):  rbuels | 2009-09-04 02:20:23 +0200
 more work on extended set of Pg auto-pk tests
 r7544@Thesaurus (orig r7541):  rbuels | 2009-09-04 02:20:32 +0200
 rewrote autoinc fetcher as a query into the pg_catalog.  all the old tests pass now, but not my new tests.  the new tests might be buggy
 r7545@Thesaurus (orig r7542):  rbuels | 2009-09-04 02:20:39 +0200
 oops, forgot to put the drop for the extended tests back in the pg tests
 r7546@Thesaurus (orig r7543):  rbuels | 2009-09-04 02:41:56 +0200
 couple of comment/documentation tweaks to pg storage driver
 r7547@Thesaurus (orig r7544):  rbuels | 2009-09-04 02:42:02 +0200
 fixed my tests
 r7548@Thesaurus (orig r7545):  rbuels | 2009-09-04 02:42:09 +0200
 clarified the POD in Pg storage driver regarding multi-schema support
 r7551@Thesaurus (orig r7548):  ribasushi | 2009-09-04 08:51:30 +0200
 Proper unconnected test
 r7554@Thesaurus (orig r7551):  ribasushi | 2009-09-04 11:26:12 +0200
 Fixes to pg test after review:
 - Move the store_column test to 60core.t
 - Streamline the select ... for update test
 - Disable all exception warnings for normal test runs

 r7555@Thesaurus (orig r7552):  ribasushi | 2009-09-04 11:56:00 +0200
 Rewrite selector using sqla
 r7562@Thesaurus (orig r7559):  rbuels | 2009-09-04 19:42:52 +0200
 moved search_path querying function from Pg storage driver into tests
 r7563@Thesaurus (orig r7560):  rbuels | 2009-09-04 19:43:00 +0200
 refactored how Pg storage driver calls sequence search, made erorror message more informative when query into pg_catalog fails
 r7564@Thesaurus (orig r7561):  rbuels | 2009-09-04 19:43:08 +0200
 tweaked pg sequence discovery error message a bit more
 r7565@Thesaurus (orig r7562):  rbuels | 2009-09-04 19:43:17 +0200
 added big block comment explaining Pg sequence discovery strategy
 r7566@Thesaurus (orig r7563):  rbuels | 2009-09-04 20:35:10 +0200
 added code to use DBD::Pg column_info to fetch column default if recent enough
 r7567@Thesaurus (orig r7564):  rbuels | 2009-09-04 20:35:18 +0200
 tweaked comment
 r7568@Thesaurus (orig r7565):  rbuels | 2009-09-04 20:35:30 +0200
 oops, DBD::Pg 2.15.1 should be included in working versions

r7572@Thesaurus (orig r7569):  ribasushi | 2009-09-04 21:32:01 +0200
Stop double-caching datetime_parser - keep it in the storage only
r7573@Thesaurus (orig r7570):  ribasushi | 2009-09-04 21:36:39 +0200
No Serialize::Storable in core
r7574@Thesaurus (orig r7571):  ribasushi | 2009-09-04 21:49:54 +0200
Changes
r7580@Thesaurus (orig r7577):  ribasushi | 2009-09-06 12:28:44 +0200
Add mysterious exception test
r7582@Thesaurus (orig r7579):  ribasushi | 2009-09-06 15:43:10 +0200
No connection - no cleanup
r7583@Thesaurus (orig r7580):  ribasushi | 2009-09-06 15:45:51 +0200
Streamline test
r7584@Thesaurus (orig r7581):  ribasushi | 2009-09-06 17:39:03 +0200
Test cleanup:
Benchmark and Data::Dumper have been in core forever
Make POD testing conditional as shown in http://use.perl.org/~Alias/journal/38822
Remove some dead cdbi test files
Stop openly giving contributors an option to override the authorcheck

r7585@Thesaurus (orig r7582):  ribasushi | 2009-09-06 17:48:32 +0200
Done long time ago
r7586@Thesaurus (orig r7583):  ribasushi | 2009-09-06 17:56:27 +0200
Release 0.08110
r7588@Thesaurus (orig r7585):  ribasushi | 2009-09-06 18:33:46 +0200
Stop eating exceptions in ::Storage::DBI::DESTROY
r7589@Thesaurus (orig r7586):  ribasushi | 2009-09-06 20:35:30 +0200
Centralize identity insert control for mssql (it seems that issuing an OFF is not necessary)
r7590@Thesaurus (orig r7587):  ribasushi | 2009-09-06 20:45:41 +0200
Clearer MSSQL error message
r7591@Thesaurus (orig r7588):  ribasushi | 2009-09-06 23:58:22 +0200
Fix mssql pod
r7592@Thesaurus (orig r7589):  ribasushi | 2009-09-07 09:06:05 +0200
Release 0.08111

Whoops this should not have committed

Release 0.08111 tag

Release 0.08111

Fix mssql pod

Clearer MSSQL error message

Centralize identity insert control for mssql (it seems that issuing an OFF is not necessary)

Stop eating exceptions in ::Storage::DBI::DESTROY

Release 0.08110

Done long time ago

Test cleanup:
Benchmark and Data::Dumper have been in core forever
Make POD testing conditional as shown in http://use.perl.org/~Alias/journal/38822
Remove some dead cdbi test files
Stop openly giving contributors an option to override the authorcheck

Streamline test

No connection - no cleanup

Add mysterious exception test

pending review by mpeppler

Changes

No Serialize::Storable in core

Stop double-caching datetime_parser - keep it in the storage only

Merge 'pg_unqualified_schema' into 'trunk'

oops, DBD::Pg 2.15.1 should be included in working versions

tweaked comment

added code to use DBD::Pg column_info to fetch column default if recent enough

added big block comment explaining Pg sequence discovery strategy

tweaked pg sequence discovery error message a bit more

refactored how Pg storage driver calls sequence search, made erorror message more informative when query into pg_catalog fails

moved search_path querying function from Pg storage driver into tests

Even better sqla

Dumper follies

Rewrite selector using sqla

Fixes to pg test after review:
- Move the store_column test to 60core.t
- Streamline the select ... for update test
- Disable all exception warnings for normal test runs