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

Merge 'storage-tweaks' into 'trunk'

Clarify test

Sometimes _prepare_for_execute may not return a set of bind vars - guard against that

join can be found without seen_join

Better testing and heuristics for subqueried _cond_for_update_delete

Fix _select_for_update/delete - bring back old code, use subqueries only when resultset attributes call for it

fix bogus test

Fix shot test

Some more die/croak conversions

Changes for count_distinct branch merge

Test for failing code related to many to many in MySQL

Added synopsis of merged oracle-tweaks branch to Changes

added info to Changes from my DateTime::InflateColumn modifications

Switch warn to carp and die to throw_exception where possible

Rename m2m warn disable envvar

Shuffle delete tests, and sanify the delete related ones

Cleanup

Merge 'oracle-tweaks' into 'trunk'

Whops, forgotten post-merge tweaks

Merge 'trunk' into 'oracle-tweaks'

r6195@Thesaurus (orig r6194):  caelum | 2009-05-09 15:46:55 +0200
added postgres default port stuff to FAQ
r6199@Thesaurus (orig r6198):  mo | 2009-05-10 17:37:16 +0200
set_$rel accepts now a $link_vals hashref like add_to_$rel does
r6202@Thesaurus (orig r6201):  ribasushi | 2009-05-10 19:57:31 +0200
Require DBICTest.pm in all tests even if it is not needed at all
r6203@Thesaurus (orig r6202):  ribasushi | 2009-05-10 20:17:09 +0200
No more 'I forgot to run perl Makefile.PL'
r6204@Thesaurus (orig r6203):  ribasushi | 2009-05-10 21:35:03 +0200
Switch the m2m method warnings from warnings::register to $ENV{DBIC_METHOD_CLOBBER_OK} = 1
r6206@Thesaurus (orig r6205):  ribasushi | 2009-05-11 12:50:54 +0200
Change the makefile to test r6202
r6218@Thesaurus (orig r6217):  ribasushi | 2009-05-12 07:02:14 +0200
Trying to untangle failing merge (whitespace change)
r6220@Thesaurus (orig r6219):  ribasushi | 2009-05-12 07:02:56 +0200
 r5757@Thesaurus (orig r5756):  arcanez | 2009-03-14 14:55:55 +0100
 created count_distinct branch
 r5758@Thesaurus (orig r5757):  arcanez | 2009-03-14 15:33:03 +0100
  * change count with group_by (distinct) to use a subquery
  * rewrite of _bind_to_sql (uses placeholders and bindvars)
  * tests for count distinct
  * fixed tests for from subquery

 r5760@Thesaurus (orig r5759):  arcanez | 2009-03-16 16:48:28 +0100
 don't remove the where clause unless we're doing distinct, it needs to be there
 r5850@Thesaurus (orig r5849):  arcanez | 2009-03-30 21:40:05 +0200
  * add more tests
  * remove old cruft
  * remove old note

 r6035@Thesaurus (orig r6034):  ribasushi | 2009-04-30 09:10:36 +0200
 Add joined count test
 r6079@Thesaurus (orig r6078):  arcanez | 2009-05-01 08:41:34 +0200
 cleanup/fix some broken tests
 r6097@Thesaurus (orig r6096):  arcanez | 2009-05-01 19:37:04 +0200
 make sure merge bind
 test for aformentioned
 TODO count_joined test for a little while

 r6105@Thesaurus (orig r6104):  arcanez | 2009-05-02 02:33:49 +0200
 remove hackish ways

 r6106@Thesaurus (orig r6105):  arcanez | 2009-05-02 03:20:04 +0200
 more fixes to tests
 r6107@Thesaurus (orig r6106):  arcanez | 2009-05-02 03:33:47 +0200
 remove DBIx::Class::Storage::DBI::MultiDistinctEmulation
 r6111@Thesaurus (orig r6110):  ribasushi | 2009-05-03 02:00:19 +0200
 Failing test without immediate fixes go to branches, not to trunk
 r6114@Thesaurus (orig r6113):  ribasushi | 2009-05-03 10:23:28 +0200
 Bump SQLA ependencies so parenthesis_significant is guaranteed to be there
 r6148@Thesaurus (orig r6147):  ribasushi | 2009-05-06 17:40:31 +0200
 Add subquery/from test by michaelr (copied from subquery branch r5742)
 r6151@Thesaurus (orig r6150):  ribasushi | 2009-05-06 17:56:07 +0200
 TODOify sqla-dependent tests
 r6152@Thesaurus (orig r6151):  arcanez | 2009-05-06 18:33:58 +0200
 fix old test using new bind vars (no more interpolating)
 r6154@Thesaurus (orig r6153):  ribasushi | 2009-05-07 00:56:40 +0200
 Rename internal function to clarify what it does
 r6171@Thesaurus (orig r6170):  ribasushi | 2009-05-07 19:53:30 +0200
 Adjust tests for the IN fixes
 r6187@Thesaurus (orig r6186):  ribasushi | 2009-05-08 23:11:43 +0200
 Final count tests
 r6188@Thesaurus (orig r6187):  arcanez | 2009-05-09 03:50:12 +0200
 rewrite DISTINCT/COUNT(DISTINCT) Cookbook entries
 r6213@Thesaurus (orig r6212):  arcanez | 2009-05-11 22:41:21 +0200
 warn/die based on { select => { distinct => { } } }
 r6214@Thesaurus (orig r6213):  arcanez | 2009-05-11 23:21:11 +0200
 use carp instead of warn
 r6216@Thesaurus (orig r6215):  arcanez | 2009-05-11 23:45:05 +0200
 make sure we get just a string
 r6217@Thesaurus (orig r6216):  arcanez | 2009-05-11 23:52:11 +0200
 oops

Merge 'count_distinct' into 'trunk'

Trying to untangle failing merge (whitespace change)

oops

make sure we get just a string

Merge 'trunk' into 'count_distinct'

r6172@mullet (orig r6171):  ribasushi | 2009-05-07 11:49:26 -0700
Cookbook cleanup
r6174@mullet (orig r6173):  ribasushi | 2009-05-08 01:13:30 -0700
Throw away some debugging code
r6175@mullet (orig r6174):  ribasushi | 2009-05-08 01:21:53 -0700
Documentation patch by nniuq
r6176@mullet (orig r6175):  plu | 2009-05-08 01:30:20 -0700
Set NLS_LANG so we  have a predictable date format when using MON
r6177@mullet (orig r6176):  ribasushi | 2009-05-08 03:15:15 -0700
Fix POD
r6179@mullet (orig r6178):  jgoulah | 2009-05-08 07:27:49 -0700
renaming rh performance test so it will show up at the end of test output
r6195@mullet (orig r6194):  caelum | 2009-05-09 06:46:55 -0700
added postgres default port stuff to FAQ
r6199@mullet (orig r6198):  mo | 2009-05-10 08:37:16 -0700
set_$rel accepts now a $link_vals hashref like add_to_$rel does
r6202@mullet (orig r6201):  ribasushi | 2009-05-10 10:57:31 -0700
Require DBICTest.pm in all tests even if it is not needed at all
r6203@mullet (orig r6202):  ribasushi | 2009-05-10 11:17:09 -0700
No more 'I forgot to run perl Makefile.PL'
r6204@mullet (orig r6203):  ribasushi | 2009-05-10 12:35:03 -0700
Switch the m2m method warnings from warnings::register to $ENV{DBIC_METHOD_CLOBBER_OK} = 1
r6206@mullet (orig r6205):  ribasushi | 2009-05-11 03:50:54 -0700
Change the makefile to test r6202

use carp instead of warn

warn/die based on { select => { distinct => { } } }

Moved DBIC::SQL::Abstract inner classes to DBIx::Class::SQLAHacks namespace to decouple 41orrible.t tests from use of DBD::Oracle in Oracle Generic driver.

Change the makefile to test r6202

Switch the m2m method warnings from warnings::register to $ENV{DBIC_METHOD_CLOBBER_OK} = 1

No more 'I forgot to run perl Makefile.PL'

Require DBICTest.pm in all tests even if it is not needed at all

set_$rel accepts now a $link_vals hashref like add_to_$rel does

added postgres default port stuff to FAQ

::DBI::Replicated - don't build pool/balancer from connect_info unless necessary

updated Changes

Merge 'trunk' into 'storage-tweaks'

r6127@Thesaurus (orig r6126):  arcanez | 2009-05-05 09:27:56 +0200
 r6037@mullet (orig r6036):  arcanez | 2009-04-30 01:24:41 -0700
 branch to work on Sybase/MSSQL subtleties
 r6038@mullet (orig r6037):  arcanez | 2009-04-30 01:27:11 -0700
 jump to ::DBI::Sybase::MSSQL if we are using MSSQL through Sybase
 r6062@mullet (orig r6061):  arcanez | 2009-04-30 14:05:24 -0700
 fixes for MSSQL via Sybase
 r6126@mullet (orig r6125):  arcanez | 2009-05-05 00:27:13 -0700
 add a line to Changes and add a CAVEAT

r6164@Thesaurus (orig r6163):  ribasushi | 2009-05-07 19:09:01 +0200
 r6115@Thesaurus (orig r6114):  plu | 2009-05-03 10:39:16 +0200
 new branch to fix $rs->update and $rs->delete using the new as_query method

 r6116@Thesaurus (orig r6115):  plu | 2009-05-03 10:52:07 +0200
 Methods update/delete on resultset use now new as_query method to updated/delete properly on joined/prefetched resultset using a subquery. Therefore some tests have been added and some have been changed as well as the warnings around $rs->update/delete have been removed. Cheers!
 r6117@Thesaurus (orig r6116):  plu | 2009-05-03 11:13:48 +0200
 Using "is" instead of "cmp_ok"
 r6160@Thesaurus (orig r6159):  ribasushi | 2009-05-07 11:58:14 +0200
 Back out skip_parens support in as_query
 r6161@Thesaurus (orig r6160):  ribasushi | 2009-05-07 19:00:48 +0200
 This test is completely borked, needs a rewrite
 r6162@Thesaurus (orig r6161):  ribasushi | 2009-05-07 19:07:19 +0200
 Temporary fix or the IN ( ( ... ) ) problem until we get proper SQLA AST (needs SQLA released with commit 6158 to work)

r6165@Thesaurus (orig r6164):  ribasushi | 2009-05-07 19:11:46 +0200
Changes, remove merged branch
r6169@Thesaurus (orig r6168):  ribasushi | 2009-05-07 19:24:54 +0200
Bump SQLA dependency so -in/-between workarounds overload properly
r6172@Thesaurus (orig r6171):  ribasushi | 2009-05-07 20:49:26 +0200
Cookbook cleanup
r6174@Thesaurus (orig r6173):  ribasushi | 2009-05-08 10:13:30 +0200
Throw away some debugging code
r6175@Thesaurus (orig r6174):  ribasushi | 2009-05-08 10:21:53 +0200
Documentation patch by nniuq
r6176@Thesaurus (orig r6175):  plu | 2009-05-08 10:30:20 +0200
Set NLS_LANG so we  have a predictable date format when using MON
r6177@Thesaurus (orig r6176):  ribasushi | 2009-05-08 12:15:15 +0200
Fix POD
r6179@Thesaurus (orig r6178):  jgoulah | 2009-05-08 16:27:49 +0200
renaming rh performance test so it will show up at the end of test output

::DBI::Replicated - fix fallback to master, test for the warning, other cleanups

::DBI::Replicated - add master_read_weight to ::Random balancer_type

Changed test of lob values from is to ok on an eq expr, to avoid a huge got/expected diagnosis.  In doing so, discovered it was testing undef vs undef!  Whoops.  Fixed.

rewrite DISTINCT/COUNT(DISTINCT) Cookbook entries

Final count tests

Merge 'trunk' into 'oracle-tweaks'

r6114@Thesaurus (orig r6113):  ribasushi | 2009-05-03 10:23:28 +0200
Bump SQLA ependencies so parenthesis_significant is guaranteed to be there
r6127@Thesaurus (orig r6126):  arcanez | 2009-05-05 09:27:56 +0200
 r6037@mullet (orig r6036):  arcanez | 2009-04-30 01:24:41 -0700
 branch to work on Sybase/MSSQL subtleties
 r6038@mullet (orig r6037):  arcanez | 2009-04-30 01:27:11 -0700
 jump to ::DBI::Sybase::MSSQL if we are using MSSQL through Sybase
 r6062@mullet (orig r6061):  arcanez | 2009-04-30 14:05:24 -0700
 fixes for MSSQL via Sybase
 r6126@mullet (orig r6125):  arcanez | 2009-05-05 00:27:13 -0700
 add a line to Changes and add a CAVEAT

r6164@Thesaurus (orig r6163):  ribasushi | 2009-05-07 19:09:01 +0200
 r6115@Thesaurus (orig r6114):  plu | 2009-05-03 10:39:16 +0200
 new branch to fix $rs->update and $rs->delete using the new as_query method

 r6116@Thesaurus (orig r6115):  plu | 2009-05-03 10:52:07 +0200
 Methods update/delete on resultset use now new as_query method to updated/delete properly on joined/prefetched resultset using a subquery. Therefore some tests have been added and some have been changed as well as the warnings around $rs->update/delete have been removed. Cheers!
 r6117@Thesaurus (orig r6116):  plu | 2009-05-03 11:13:48 +0200
 Using "is" instead of "cmp_ok"
 r6160@Thesaurus (orig r6159):  ribasushi | 2009-05-07 11:58:14 +0200
 Back out skip_parens support in as_query
 r6161@Thesaurus (orig r6160):  ribasushi | 2009-05-07 19:00:48 +0200
 This test is completely borked, needs a rewrite
 r6162@Thesaurus (orig r6161):  ribasushi | 2009-05-07 19:07:19 +0200
 Temporary fix or the IN ( ( ... ) ) problem until we get proper SQLA AST (needs SQLA released with commit 6158 to work)

r6165@Thesaurus (orig r6164):  ribasushi | 2009-05-07 19:11:46 +0200
Changes, remove merged branch
r6169@Thesaurus (orig r6168):  ribasushi | 2009-05-07 19:24:54 +0200
Bump SQLA dependency so -in/-between workarounds overload properly
r6172@Thesaurus (orig r6171):  ribasushi | 2009-05-07 20:49:26 +0200
Cookbook cleanup
r6174@Thesaurus (orig r6173):  ribasushi | 2009-05-08 10:13:30 +0200
Throw away some debugging code
r6175@Thesaurus (orig r6174):  ribasushi | 2009-05-08 10:21:53 +0200
Documentation patch by nniuq
r6176@Thesaurus (orig r6175):  plu | 2009-05-08 10:30:20 +0200
Set NLS_LANG so we  have a predictable date format when using MON
r6177@Thesaurus (orig r6176):  ribasushi | 2009-05-08 12:15:15 +0200
Fix POD
r6179@Thesaurus (orig r6178):  jgoulah | 2009-05-08 16:27:49 +0200
renaming rh performance test so it will show up at the end of test output

support ::DBI::Replicated opts in connect_info

Remove redundant var

Refactored to call _{inflate_to,deflate_from}_datetime through a _flate_or_fallback wrapper handling a parser's lack of support for the requested type.

renaming rh performance test so it will show up at the end of test output

Fix POD

Set NLS_LANG so we  have a predictable date format when using MON

Documentation patch by nniuq

Throw away some debugging code

Cookbook cleanup

Adjust tests for the IN fixes

Merge 'trunk' into 'count_distinct'

r6164@Thesaurus (orig r6163):  ribasushi | 2009-05-07 19:09:01 +0200
 r6115@Thesaurus (orig r6114):  plu | 2009-05-03 10:39:16 +0200
 new branch to fix $rs->update and $rs->delete using the new as_query method

 r6116@Thesaurus (orig r6115):  plu | 2009-05-03 10:52:07 +0200
 Methods update/delete on resultset use now new as_query method to updated/delete properly on joined/prefetched resultset using a subquery. Therefore some tests have been added and some have been changed as well as the warnings around $rs->update/delete have been removed. Cheers!
 r6117@Thesaurus (orig r6116):  plu | 2009-05-03 11:13:48 +0200
 Using "is" instead of "cmp_ok"
 r6160@Thesaurus (orig r6159):  ribasushi | 2009-05-07 11:58:14 +0200
 Back out skip_parens support in as_query
 r6161@Thesaurus (orig r6160):  ribasushi | 2009-05-07 19:00:48 +0200
 This test is completely borked, needs a rewrite
 r6162@Thesaurus (orig r6161):  ribasushi | 2009-05-07 19:07:19 +0200
 Temporary fix or the IN ( ( ... ) ) problem until we get proper SQLA AST (needs SQLA released with commit 6158 to work)

r6165@Thesaurus (orig r6164):  ribasushi | 2009-05-07 19:11:46 +0200
Changes, remove merged branch
r6169@Thesaurus (orig r6168):  ribasushi | 2009-05-07 19:24:54 +0200
Bump SQLA dependency so -in/-between workarounds overload properly

Bump SQLA dependency so -in/-between workarounds overload properly

Changes, remove merged branch

Merge 'fix-update-and-delete-as_query' into 'trunk'

Temporary fix or the IN ( ( ... ) ) problem until we get proper SQLA AST (needs SQLA released with commit 6158 to work)

This test is completely borked, needs a rewrite

Back out skip_parens support in as_query

Rename internal function to clarify what it does

fix old test using new bind vars (no more interpolating)

TODOify sqla-dependent tests

Merge 'trunk' into 'count_distinct'

r6110@Thesaurus (orig r6109):  plu | 2009-05-02 21:11:40 +0200
Failing testcase for using subselect (as_query)
r6111@Thesaurus (orig r6110):  ribasushi | 2009-05-03 02:00:19 +0200
Failing test without immediate fixes go to branches, not to trunk
r6114@Thesaurus (orig r6113):  ribasushi | 2009-05-03 10:23:28 +0200
Bump SQLA ependencies so parenthesis_significant is guaranteed to be there
r6127@Thesaurus (orig r6126):  arcanez | 2009-05-05 09:27:56 +0200
 r6037@mullet (orig r6036):  arcanez | 2009-04-30 01:24:41 -0700
 branch to work on Sybase/MSSQL subtleties
 r6038@mullet (orig r6037):  arcanez | 2009-04-30 01:27:11 -0700
 jump to ::DBI::Sybase::MSSQL if we are using MSSQL through Sybase
 r6062@mullet (orig r6061):  arcanez | 2009-04-30 14:05:24 -0700
 fixes for MSSQL via Sybase
 r6126@mullet (orig r6125):  arcanez | 2009-05-05 00:27:13 -0700
 add a line to Changes and add a CAVEAT

Merge 'subquery' into 'count_distinct'

Add subquery/from test by michaelr (copied from subquery branch r5742)

::DBI:Replicated - merge connect_info from master to replicants

Replicated - fixup types and namespace::clean

fix ::DBI::Replicated::all_storages

Added self to contributors; clarified comment on :ora_types imports.

Merge 'sybase_mssql' into 'trunk'

r6037@mullet (orig r6036):  arcanez | 2009-04-30 01:24:41 -0700
branch to work on Sybase/MSSQL subtleties
r6038@mullet (orig r6037):  arcanez | 2009-04-30 01:27:11 -0700
jump to ::DBI::Sybase::MSSQL if we are using MSSQL through Sybase
r6062@mullet (orig r6061):  arcanez | 2009-04-30 14:05:24 -0700
fixes for MSSQL via Sybase
r6126@mullet (orig r6125):  arcanez | 2009-05-05 00:27:13 -0700
add a line to Changes and add a CAVEAT

add a line to Changes and add a CAVEAT

Merge 'trunk' into 'sybase_mssql'

r6110@mullet (orig r6109):  plu | 2009-05-02 12:11:40 -0700
Failing testcase for using subselect (as_query)
r6111@mullet (orig r6110):  ribasushi | 2009-05-02 17:00:19 -0700
Failing test without immediate fixes go to branches, not to trunk
r6114@mullet (orig r6113):  ribasushi | 2009-05-03 01:23:28 -0700
Bump SQLA ependencies so parenthesis_significant is guaranteed to be there

::Replicated - test hashref for connect_replicants and croak on coderef, switch to MX::Types, make test less noisy

support hashrefs for connect_replicants

Re-added last_updated_at to create table statement.

Proper support for timestamp inflation.  Added last_updated_at to DBICTest::Schema::Track as a date by default, initialized in sqlite loader, redefined to timestamp for Oracle tests.

Using "is" instead of "cmp_ok"

Methods update/delete on resultset use now new as_query method to updated/delete properly on joined/prefetched resultset using a subquery. Therefore some tests have been added and some have been changed as well as the warnings around $rs->update/delete have been removed. Cheers!

Bump SQLA ependencies so parenthesis_significant is guaranteed to be there

Bump SQLA ependencies so parenthesis_significant is guaranteed to be there

Support for saving CLOB and BLOB types in Oracle.

Failing test without immediate fixes go to branches, not to trunk

Failing test without immediate fixes go to branches, not to trunk

Failing testcase for using subselect (as_query)

Merge 'trunk' into 'sybase_mssql'

r6039@mullet (orig r6038):  ribasushi | 2009-04-30 01:28:04 -0700
Release 0.08102

Merge 'trunk' into 'count_distinct'

r5987@mullet (orig r5986):  castaway | 2009-04-25 02:59:28 -0700
Docs on using multiple db schemas (from abraxxa maybe, I forget)
Docs on setting result_class from agaran

r5988@mullet (orig r5987):  ribasushi | 2009-04-27 01:03:58 -0700
Refactor the (almost obsolete) DBD::SQLite check not to fork() within win32
r5989@mullet (orig r5988):  ribasushi | 2009-04-27 01:05:43 -0700
Whops, we don't exit under win32
r5990@mullet (orig r5989):  ribasushi | 2009-04-27 01:14:38 -0700
More AUTHOR dependencies pulled from Replicated.pm
Up MI version to 0.79 (not going further as auto_install is broken again, need a permanent solution)
Reorder configure_requires() as per Alias
Remove Meta_TupleKeys munging as per Alias
r5991@mullet (orig r5990):  ribasushi | 2009-04-27 02:11:15 -0700
Release 0.08101
r5995@mullet (orig r5994):  arcanez | 2009-04-28 00:05:53 -0700
change from DB::Schema to My::Schema
add ::Result:: as necessary (load_namespaces instead of load_classes)

r5996@mullet (orig r5995):  ribasushi | 2009-04-28 02:12:33 -0700
Fix a todo and some minor test adjustments
r5997@mullet (orig r5996):  ribasushi | 2009-04-28 02:35:31 -0700
Bogus test dependency
r6003@mullet (orig r6002):  arcanez | 2009-04-28 13:20:42 -0700
fix for columns => [ ... ] issue
r6014@mullet (orig r6013):  arcanez | 2009-04-28 16:16:48 -0700
test for prefetch by columns
r6016@mullet (orig r6015):  ribasushi | 2009-04-29 01:22:16 -0700
test and patch for failing mini-prefetch via columns (arcanez++)
r6017@mullet (orig r6016):  ribasushi | 2009-04-29 01:23:25 -0700
Add two TODOs by arcanez (not sure about those)
r6018@mullet (orig r6017):  ribasushi | 2009-04-29 01:24:20 -0700
Add TODOs illustrating a problem with as_query
r6019@mullet (orig r6018):  arcanez | 2009-04-29 05:47:03 -0700
untodo/fix test
r6020@mullet (orig r6019):  ribasushi | 2009-04-29 06:39:27 -0700
Strip out inane todo
r6021@mullet (orig r6020):  ribasushi | 2009-04-29 06:40:03 -0700
Forgotten debugging
r6022@mullet (orig r6021):  ribasushi | 2009-04-29 06:41:37 -0700
Fixes to massive breakage introduced by 5948  ribasushi--
r6023@mullet (orig r6022):  ribasushi | 2009-04-29 06:47:23 -0700
Changes
r6024@mullet (orig r6023):  jasonmay | 2009-04-29 08:18:11 -0700
allow multiple classless sources to be registered without dying

r6025@mullet (orig r6024):  jasonmay | 2009-04-29 09:47:26 -0700
add jasonmay to list of contributors

r6034@mullet (orig r6033):  ribasushi | 2009-04-29 23:53:51 -0700
Add tests for r6002
r6036@mullet (orig r6035):  ribasushi | 2009-04-30 01:12:23 -0700
Revert addition of superficial relationship in r5886
r6039@mullet (orig r6038):  ribasushi | 2009-04-30 01:28:04 -0700
Release 0.08102

remove DBIx::Class::Storage::DBI::MultiDistinctEmulation

more fixes to tests

remove hackish ways

make sure merge bind
test for aformentioned
TODO count_joined test for a little while

cleanup/fix some broken tests

fixes for MSSQL via Sybase

Release 0.08102

jump to ::DBI::Sybase::MSSQL if we are using MSSQL through Sybase

branch to work on Sybase/MSSQL subtleties

Revert addition of superficial relationship in r5886

Add joined count test

Add tests for r6002

add jasonmay to list of contributors