Merge 'trunk' into 'sybase'

r5334@hlagh (orig r6501):  ribasushi | 2009-06-04 06:38:21 -0700
Really delegate counting to the storage class - now we have either a grouped count or a regular one

Really delegate counting to the storage class - now we have either a grouped count or a regular one

sybase limit count without offset now works

Merge 'trunk' into 'sybase'

r5325@hlagh (orig r6495):  ribasushi | 2009-06-04 00:35:25 -0700
Move relationship tests around
r5328@hlagh (orig r6498):  ribasushi | 2009-06-04 02:52:18 -0700
Delegate actual counting to the storage class

Delegate actual counting to the storage class

Move relationship tests around

update Sybase docs

test sybase SMALLDATETIME inflation

fix base for mssql (can't be a sybase anymore)

fix Sybase DT stuff and storage bases

Merge 'trunk' into 'sybase'

r4349@hlagh (orig r6389):  ribasushi | 2009-05-23 13:59:17 -0700
Extra test and count fixes for prefetch + distinct
r4350@hlagh (orig r6390):  caelum | 2009-05-23 14:04:01 -0700
minor clean up
r4351@hlagh (orig r6391):  ribasushi | 2009-05-23 14:37:19 -0700
Make sure MultiColumnIn quotes column names while munging literal sql
Tested by running t/resultset/update_delete against pg with quote char " and name sep .
r4352@hlagh (orig r6392):  ribasushi | 2009-05-23 15:47:40 -0700
update changes
r4353@hlagh (orig r6393):  ribasushi | 2009-05-23 15:50:08 -0700
Extend distinct deprecation tests and clarify warnings
r4355@hlagh (orig r6395):  ribasushi | 2009-05-24 01:47:03 -0700
MSSQL through ODBC does not like unfinished statements - make sure we finish the scope identity retrieval
(This worked before because of the automatic retry on exception, essentially running any select after insert twice)
r4356@hlagh (orig r6396):  ribasushi | 2009-05-24 02:42:01 -0700
Make sure we do not clobber search attributes when using subqueries
r4357@hlagh (orig r6397):  ribasushi | 2009-05-24 02:43:32 -0700
SUPER is so last century
r4358@hlagh (orig r6398):  ribasushi | 2009-05-24 05:12:39 -0700
Deprecate ::DBI::Sybase::MSSQL
r4359@hlagh (orig r6399):  ribasushi | 2009-05-24 06:00:50 -0700
eol adjustments
r4360@hlagh (orig r6400):  ribasushi | 2009-05-24 06:07:45 -0700
Switch trunk to native eol-style
r4361@hlagh (orig r6401):  ribasushi | 2009-05-24 06:35:07 -0700
Failing test about warnings triggered in SQLA::Limit when using a subquery
r4365@hlagh (orig r6405):  ribasushi | 2009-05-24 13:53:15 -0700
Switch around inheritance of MSSQL drivers, remove some duplicate code
r4366@hlagh (orig r6406):  caelum | 2009-05-24 16:49:17 -0700
fix double connect for ODBC/MSSQL
r4367@hlagh (orig r6407):  caelum | 2009-05-24 16:53:12 -0700
added test to make sure only one connection to ODBC/MSSQL is made
r4520@hlagh (orig r6410):  ribasushi | 2009-05-24 23:48:38 -0700
Factor out the order_by sqlahacks resolver
r4521@hlagh (orig r6411):  ribasushi | 2009-05-25 00:42:45 -0700
Move the DB2 Limit syntax setting into the storage class
r4522@hlagh (orig r6412):  ribasushi | 2009-05-25 00:47:01 -0700
Forgotten podcoverage override
r4523@hlagh (orig r6413):  ribasushi | 2009-05-25 02:11:01 -0700
Define how Top limit emulation should behave
r5249@hlagh (orig r6418):  ribasushi | 2009-05-25 07:53:13 -0700
Actually don't need this anymore
r5251@hlagh (orig r6420):  tomboh | 2009-05-25 09:33:46 -0700
Small documentation improvement:  link to a documented method.
r5253@hlagh (orig r6422):  ribasushi | 2009-05-25 10:42:02 -0700
This method does not exist anymore
r5254@hlagh (orig r6426):  ribasushi | 2009-05-25 23:15:34 -0700
TODOified test for RT#40701
r5256@hlagh (orig r6428):  ash | 2009-05-26 07:17:44 -0700
Fix 'timestamp with time zone' columns for IC::DT inflation

r5258@hlagh (orig r6430):  ribasushi | 2009-05-26 07:29:55 -0700
 r6415@Thesaurus (orig r6414):  ribasushi | 2009-05-25 11:18:05 +0200
 Yet another branch to attempt a top fix
 r6416@Thesaurus (orig r6415):  ribasushi | 2009-05-25 11:24:32 +0200
 The Top limit emulation bundled with SQLA::Limit assumes that the limited resultset will be _always_ sorted. In order to fix this, I reimplemented _Top in SQLAHacks with a slight modification. Now the original order_by is passed to the outside of the nested select block, while order_up/down are calculated either based on the original order_by, or if one is not present an order by all PKs is attempted.
 Since I do not have access to $rsrc in SQLA, I pass the list of PKs as an extra group_by hash entry. This appears to be rather safe, and besides we already pollute order_by with group_by and having (which seems to work rather well).
 The only thing I am unsure about is the need for _gen_virtual_order(). Initially I was going to generate the pk list, only if we use the Top limit. Then it turned out there is no limit dialect before we connect, so I commented it out. Now all it does is check for a limit condition and returns the PK list. Is this necessary at all?
 r6417@Thesaurus (orig r6416):  ribasushi | 2009-05-25 16:08:40 +0200
 Shoot another Top problem, move test from top_limit_tweaks branch and delete
 r6420@Thesaurus (orig r6419):  ribasushi | 2009-05-25 17:45:33 +0200
 Too much logic for no benefit - always populate _virtual_order_by

r5260@hlagh (orig r6432):  ribasushi | 2009-05-26 07:36:55 -0700
Fix test skip message
r5261@hlagh (orig r6433):  ribasushi | 2009-05-26 08:20:57 -0700
Minor fixes of the return value of rs->update/delete
r5262@hlagh (orig r6434):  ribasushi | 2009-05-26 11:49:49 -0700
fix comments
r5263@hlagh (orig r6435):  ribasushi | 2009-05-26 12:28:49 -0700
Attempt to reproduce reported mysql error (failed) - fixed another bug in ResultSetColumn along the way
r5264@hlagh (orig r6436):  ribasushi | 2009-05-26 13:02:29 -0700
Release 0.08103
r5288@hlagh (orig r6446):  ribasushi | 2009-05-28 01:20:57 -0700
Commit rather useless but already written mysql test extension
r5289@hlagh (orig r6447):  ribasushi | 2009-05-28 04:02:22 -0700
Fix multiprefetch warning - we can now count properly
r5290@hlagh (orig r6456):  ribasushi | 2009-05-29 22:40:24 -0700
Patch + test for more informative exceptions on load_namespace a non-rs class
r5291@hlagh (orig r6457):  ribasushi | 2009-05-30 00:34:20 -0700
Add better error reporting on bulk_insert (ash++)
r5300@hlagh (orig r6469):  ribasushi | 2009-05-30 10:46:09 -0700
populate() fix and Changes
r5301@hlagh (orig r6470):  ribasushi | 2009-05-31 00:37:37 -0700
M::I 0.89 finally resolves all provlems with auto_install
r5302@hlagh (orig r6471):  ribasushi | 2009-05-31 00:42:35 -0700
Throw away the makefile SQLite test - it served its purpose
r5303@hlagh (orig r6472):  ribasushi | 2009-05-31 02:24:15 -0700
There is a saner way to write out resources
r5304@hlagh (orig r6473):  ribasushi | 2009-05-31 07:07:18 -0700
Last set of Makefile.PL optimizations
r5305@hlagh (orig r6474):  ribasushi | 2009-06-01 03:24:41 -0700
deploy-related pod fixes
r5307@hlagh (orig r6476):  ribasushi | 2009-06-01 07:40:22 -0700
 r6462@Thesaurus (orig r6461):  mo | 2009-05-30 11:06:54 +0200
 order_by tests

r5308@hlagh (orig r6477):  ribasushi | 2009-06-01 07:40:30 -0700
 r6463@Thesaurus (orig r6462):  ribasushi | 2009-05-30 16:54:37 +0200
 TODOify some of the order with bind tests

r5309@hlagh (orig r6478):  ribasushi | 2009-06-01 07:40:41 -0700
 r6464@Thesaurus (orig r6463):  ribasushi | 2009-05-30 16:55:37 +0200
 Restructure bind tests

r5310@hlagh (orig r6479):  ribasushi | 2009-06-01 07:40:50 -0700
 r6465@Thesaurus (orig r6464):  ribasushi | 2009-05-30 17:15:57 +0200
 Greatly simplify _order_by override to fallback on new SQLA

r5311@hlagh (orig r6480):  ribasushi | 2009-06-01 07:40:56 -0700
 r6467@Thesaurus (orig r6466):  ribasushi | 2009-05-30 19:13:23 +0200
 Evil hack to make Carp::Clan work throughout SQLA as well

r5312@hlagh (orig r6481):  ribasushi | 2009-06-01 07:41:42 -0700
 r6468@Thesaurus (orig r6467):  ribasushi | 2009-05-30 19:17:02 +0200
 Add changes

r5313@hlagh (orig r6482):  ribasushi | 2009-06-01 07:41:49 -0700

r5315@hlagh (orig r6484):  ribasushi | 2009-06-01 07:49:09 -0700
Fix fallout from another botched merge (I suck, part 2)
r5317@hlagh (orig r6486):  ribasushi | 2009-06-03 01:14:51 -0700
Require a recent version of Date::Simple during CDBI tests

add missing files to sybase branch

resolve conflict

Require a recent version of Date::Simple during CDBI tests

Fix fallout from another botched merge (I suck, part 2)

Merge 'order_by_refactor' into 'trunk'

deploy-related pod fixes

Last set of Makefile.PL optimizations

There is a saner way to write out resources

Throw away the makefile SQLite test - it served its purpose

M::I 0.89 finally resolves all provlems with auto_install

populate() fix and Changes

Add changes

Evil hack to make Carp::Clan work throughout SQLA as well

Greatly simplify _order_by override to fallback on new SQLA

Restructure bind tests

TODOify some of the order with bind tests

order_by tests

Add better error reporting on bulk_insert (ash++)

Patch + test for more informative exceptions on load_namespace a non-rs class

Fix multiprefetch warning - we can now count properly

Commit rather useless but already written mysql test extension

Release 0.08103

Attempt to reproduce reported mysql error (failed) - fixed another bug in ResultSetColumn along the way

fix comments

Minor fixes of the return value of rs->update/delete

Fix test skip message

Merge 'top_limit_altfix' into 'trunk'

Fix 'timestamp with time zone' columns for IC::DT inflation

TODOified test for RT#40701

This method does not exist anymore

Small documentation improvement:  link to a documented method.

Too much logic for no benefit - always populate _virtual_order_by

Actually don't need this anymore

Merge 'top_limit_tweaks' into 'top_limit_altfix'

Shoot another Top problem, move test from top_limit_tweaks branch and delete

The Top limit emulation bundled with SQLA::Limit assumes that the limited resultset will be _always_ sorted. In order to fix this, I reimplemented _Top in SQLAHacks with a slight modification. Now the original order_by is passed to the outside of the nested select block, while order_up/down are calculated either based on the original order_by, or if one is not present an order by all PKs is attempted.
Since I do not have access to $rsrc in SQLA, I pass the list of PKs as an extra group_by hash entry. This appears to be rather safe, and besides we already pollute order_by with group_by and having (which seems to work rather well).
The only thing I am unsure about is the need for _gen_virtual_order(). Initially I was going to generate the pk list, only if we use the Top limit. Then it turned out there is no limit dialect before we connect, so I commented it out. Now all it does is check for a limit condition and returns the PK list. Is this necessary at all?

Define how Top limit emulation should behave

Forgotten podcoverage override

Move the DB2 Limit syntax setting into the storage class

Factor out the order_by sqlahacks resolver

added test to make sure only one connection to ODBC/MSSQL is made

fix double connect for ODBC/MSSQL

Switch around inheritance of MSSQL drivers, remove some duplicate code

Failing test about warnings triggered in SQLA::Limit when using a subquery

eol adjustments

Deprecate ::DBI::Sybase::MSSQL

SUPER is so last century

Make sure we do not clobber search attributes when using subqueries

MSSQL through ODBC does not like unfinished statements - make sure we finish the scope identity retrieval
(This worked before because of the automatic retry on exception, essentially running any select after insert twice)

try not to fuck mssql with the sybase crap

Extend distinct deprecation tests and clarify warnings

update changes

Make sure MultiColumnIn quotes column names while munging literal sql
Tested by running t/resultset/update_delete against pg with quote char " and name sep .

minor clean up

Extra test and count fixes for prefetch + distinct

Merge 'trunk' into 'sybase'

revert last Sybase changes

hide internal Sybase classes from PAUSE

Sybase bindvar and IC::DT support

Add storage component of multipk resultset update/delete for multicolumn IN capable rdbms
Switch Pg to that (tested via DBICTEST_DSN)

Tests and test schema adjustments for resultset update/delete

Sybase autopk, and a test, no limit support yet

Refactor rs_update_delete (too many methods for no reason)
Make mysql use _per_row_update_delete by default as this is the only thing it understands: http://dev.mysql.com/doc/refman/5.0/en/subquery-errors.html

Make distinct calculate columns *after* prefetch has been resolved. Tests to come

::Replicated -- check for master rebless and reapply role if necessary

duh

Some attributes require a grouped count subquery, some - just a subquery. Differentiate properly

Extend DB2 test

fix limit for DB2

comment addition

add some support for trying to determine storage driver without being connected

Duh, no point of having ordered subqueries while counting, Debolaz++

Clarify usage of _resolved_attrs by adding the explicit _resolved_attrs_copy
Clarify code in ResultSetColumn

on_connect_do now accepts a single string like it does an arrayref (patch by prema)

Switch as_query testing to direct specification in is_same_sql_bind

POD fixes

fix test for people with spaces in path

fix for sql functions in group_by

update to Changes

More explicit attribute deletion/detection - a lof of this is probably redundant, as _resoved_attrs() should flatten them away (i.e. prefetch to collapse and distinct to group_by). Nevertheless keep them there for explcetness ( delete() is cheap )

Make sure we always work with resolved rs attributes while counting (and test)

Add explicit grouping for rs update/delete operations if the parameters warrant it (tests coming tonight)

Bring back the old code from resolve_prefetch so ash's code can work
Add tests to make sure we throw it away in 09

detect/purge having bind for compeleteness

Make joined rs counts backwards compatible - we do not collapse a result exploded by a has_many join unless it is explicitly requested by distinct => 1, OR unless we have collapse set which implies prefetch

add test for distinct result of sql function

add mysql lost password faq

Tiny POD formatting fix

Simplify __count() - explicitly specify which attributes to copy, instead of deleting a whole bunch of them

Make podcoverage happy (deprecated, undocumented methods)