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)

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

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)

Adjust dependencies - most of them pull in bugfixes

Merge 'joined_count' into 'trunk'

Hide resultsource methods that should have never been documented

Make sure pg test can restart after half-way failures

added test for total_entries on a paged resultset

Make select always equal group by on count subqueries
Make sure offset does not increment boundlessly on every _resolved_attrs run

I think I got it right this time

doc patch per amiri

I contributed!

Merge 'trunk' into 'joined_count'

cleanup for ribasushi

Merge 'diamond_relationships' into 'trunk'

Test to show top limit grouping problem

Merge 'trunk' into 'top_limit_tweaks'

r6304@Thesaurus (orig r6303):  frew | 2009-05-18 21:43:48 +0200
Test to show problem
r6305@Thesaurus (orig r6304):  ribasushi | 2009-05-18 22:12:35 +0200
Failing tests go to branches
r6306@Thesaurus (orig r6305):  ribasushi | 2009-05-18 22:14:26 +0200
Require bugfixed sqla

Require bugfixed sqla

Failing tests go to branches

Test to show problem

pager optimization

Merge 'trunk' into 'joined_count'

r6290@Thesaurus (orig r6289):  ribasushi | 2009-05-17 00:06:40 +0200
Rewrite SqlMakerTest to fully depend on SQLA::Test
Allow is_same_sql_bind to accept an arrayrefref instead of any pair of $sql, \@bind
r6299@Thesaurus (orig r6298):  frew | 2009-05-18 18:16:44 +0200
Fix order by clauses for MSSQL

Fix order by clauses for MSSQL

Add a comment about a small resolve_join omission

Fixes for the diamond-relationship prefetch/join problem
The core of the issue was that resolve_prefetch calculated duplicate join alias numbers separate from resolve_join
In order to solve this, now the only join alias calculation happens in resolve_join (with prefetch being always merged as extra joins), and each join arrayref in from is labeled with the full relationship chain from me to the particular join. Then resolve_prefetch has to walk this chain and pull the necessary alias in order to generate the correct select

Merge 'trunk' into 'diamond_relationships'

r6255@Thesaurus (orig r6254):  ribasushi | 2009-05-14 12:12:44 +0200
Sometimes _prepare_for_execute may not return a set of bind vars - guard against that
r6256@Thesaurus (orig r6255):  ribasushi | 2009-05-14 12:14:19 +0200
Clarify test
r6264@Thesaurus (orig r6263):  ribasushi | 2009-05-14 15:40:19 +0200
 r6121@Thesaurus (orig r6120):  caelum | 2009-05-04 17:13:22 +0200
 Making new branch for storage tweaking
 r6122@Thesaurus (orig r6121):  caelum | 2009-05-04 20:07:47 +0200
 support hashrefs for connect_replicants
 r6123@Thesaurus (orig r6122):  caelum | 2009-05-04 23:07:43 +0200
 ::Replicated - test hashref for connect_replicants and croak on coderef, switch to MX::Types, make test less noisy
 r6143@Thesaurus (orig r6142):  caelum | 2009-05-06 05:13:56 +0200
 fix ::DBI::Replicated::all_storages
 r6144@Thesaurus (orig r6143):  caelum | 2009-05-06 05:25:04 +0200
 Replicated - fixup types and namespace::clean
 r6147@Thesaurus (orig r6146):  caelum | 2009-05-06 15:29:39 +0200
 ::DBI:Replicated - merge connect_info from master to replicants
 r6184@Thesaurus (orig r6183):  caelum | 2009-05-08 18:08:29 +0200
 support ::DBI::Replicated opts in connect_info
 r6190@Thesaurus (orig r6189):  caelum | 2009-05-09 05:31:15 +0200
 ::DBI::Replicated - add master_read_weight to ::Random balancer_type
 r6191@Thesaurus (orig r6190):  caelum | 2009-05-09 12:50:25 +0200
 ::DBI::Replicated - fix fallback to master, test for the warning, other cleanups
 r6193@Thesaurus (orig r6192):  caelum | 2009-05-09 13:52:52 +0200
 updated Changes
 r6194@Thesaurus (orig r6193):  caelum | 2009-05-09 14:21:44 +0200
 ::DBI::Replicated - don't build pool/balancer from connect_info unless necessary

r6268@Thesaurus (orig r6267):  caelum | 2009-05-15 04:04:12 +0200
minor replication changes - use a real hash merge, clarify master_read_weight, really done with this now.
r6272@Thesaurus (orig r6271):  abraxxa | 2009-05-15 13:45:54 +0200
added Static sub-classing DBIx::Class result classes section to the cookbook

r6277@Thesaurus (orig r6276):  ribasushi | 2009-05-16 00:46:00 +0200
Optimize some Ordered.pm code
r6278@Thesaurus (orig r6277):  ribasushi | 2009-05-16 08:02:18 +0200
Cleanup tests
r6280@Thesaurus (orig r6279):  ribasushi | 2009-05-16 09:30:05 +0200
Not sure what this part of the test is for, but it breaks custom resultsets, and the test passes without it. Removing as a possible remnant of an ancient civilization
r6281@Thesaurus (orig r6280):  ribasushi | 2009-05-16 09:33:24 +0200
Add default resultclass/resultsetclass to the entire test schema, as I am tired of typing extra shit for debugging purposes
r6282@Thesaurus (orig r6281):  ribasushi | 2009-05-16 09:34:21 +0200
Now we can do diag $rs->hri_dump, ain't that nice
r6290@Thesaurus (orig r6289):  ribasushi | 2009-05-17 00:06:40 +0200
Rewrite SqlMakerTest to fully depend on SQLA::Test
Allow is_same_sql_bind to accept an arrayrefref instead of any pair of $sql, \@bind

Rewrite SqlMakerTest to fully depend on SQLA::Test
Allow is_same_sql_bind to accept an arrayrefref instead of any pair of $sql, \@bind

Test and fixed for paged grouped count

Merge 'trunk' into 'joined_count'

r6264@Thesaurus (orig r6263):  ribasushi | 2009-05-14 15:40:19 +0200
 r6121@Thesaurus (orig r6120):  caelum | 2009-05-04 17:13:22 +0200
 Making new branch for storage tweaking
 r6122@Thesaurus (orig r6121):  caelum | 2009-05-04 20:07:47 +0200
 support hashrefs for connect_replicants
 r6123@Thesaurus (orig r6122):  caelum | 2009-05-04 23:07:43 +0200
 ::Replicated - test hashref for connect_replicants and croak on coderef, switch to MX::Types, make test less noisy
 r6143@Thesaurus (orig r6142):  caelum | 2009-05-06 05:13:56 +0200
 fix ::DBI::Replicated::all_storages
 r6144@Thesaurus (orig r6143):  caelum | 2009-05-06 05:25:04 +0200
 Replicated - fixup types and namespace::clean
 r6147@Thesaurus (orig r6146):  caelum | 2009-05-06 15:29:39 +0200
 ::DBI:Replicated - merge connect_info from master to replicants
 r6184@Thesaurus (orig r6183):  caelum | 2009-05-08 18:08:29 +0200
 support ::DBI::Replicated opts in connect_info
 r6190@Thesaurus (orig r6189):  caelum | 2009-05-09 05:31:15 +0200
 ::DBI::Replicated - add master_read_weight to ::Random balancer_type
 r6191@Thesaurus (orig r6190):  caelum | 2009-05-09 12:50:25 +0200
 ::DBI::Replicated - fix fallback to master, test for the warning, other cleanups
 r6193@Thesaurus (orig r6192):  caelum | 2009-05-09 13:52:52 +0200
 updated Changes
 r6194@Thesaurus (orig r6193):  caelum | 2009-05-09 14:21:44 +0200
 ::DBI::Replicated - don't build pool/balancer from connect_info unless necessary

r6268@Thesaurus (orig r6267):  caelum | 2009-05-15 04:04:12 +0200
minor replication changes - use a real hash merge, clarify master_read_weight, really done with this now.
r6272@Thesaurus (orig r6271):  abraxxa | 2009-05-15 13:45:54 +0200
added Static sub-classing DBIx::Class result classes section to the cookbook

r6277@Thesaurus (orig r6276):  ribasushi | 2009-05-16 00:46:00 +0200
Optimize some Ordered.pm code
r6278@Thesaurus (orig r6277):  ribasushi | 2009-05-16 08:02:18 +0200
Cleanup tests
r6280@Thesaurus (orig r6279):  ribasushi | 2009-05-16 09:30:05 +0200
Not sure what this part of the test is for, but it breaks custom resultsets, and the test passes without it. Removing as a possible remnant of an ancient civilization
r6281@Thesaurus (orig r6280):  ribasushi | 2009-05-16 09:33:24 +0200
Add default resultclass/resultsetclass to the entire test schema, as I am tired of typing extra shit for debugging purposes
r6282@Thesaurus (orig r6281):  ribasushi | 2009-05-16 09:34:21 +0200
Now we can do diag $rs->hri_dump, ain't that nice

Now we can do diag $rs->hri_dump, ain't that nice

Add default resultclass/resultsetclass to the entire test schema, as I am tired of typing extra shit for debugging purposes

Not sure what this part of the test is for, but it breaks custom resultsets, and the test passes without it. Removing as a possible remnant of an ancient civilization

Rename tests

Cleanup tests

Optimize some Ordered.pm code

A workable fix for the resultset multicol update/delete debacle - by default fallback to per-row deletions, with the ability to overide this behavior for various storage drivers

added Static sub-classing DBIx::Class result classes section to the cookbook

Add failing multikey rs delete (and by implication update) test

minor replication changes - use a real hash merge, clarify master_read_weight, really done with this now.

Merge 'storage-tweaks' into 'trunk'

Questionable test changes - need a review

Straightforward test changes