make Ordered support skimming (eg. TOP) limits

Skimming limits like TOP now require a unique key in the order_by clause
for stability, but Ordered only uses the position column in its order_by
clauses, while Result classes using Ordered generally have a compound
unique constraint on the grouping column and position column.

Append the grouping column to the end of the Ordered order_by clauses so
that it uses the full unique constraint and satisfy the skimming limit
stability requirements. This is harmless as the rows will still be
ordered on the position column.

Fix more $schema leaks in the SQLT DBIC Parser (AUGHHHHH!!!!)

Add a TODO test for a very elaborate stack leak scenario

Fix fallout from a807d012 (folks please run your podtests when hacking POD)

- Stray =end in DBIC/DB.pm
- Inadvertently hidden documentation of txn_scope_guard
  (the =for comment never applied to the POD-block previously)

Minor messing with syntactically incorrect POD (no =headX blocks inside =begin, and =for is a one-paragraph block)

Simplify code of GenericSubQ limit dialect

Some cleanups and code dedup of Top and FetchFirst limit dialects

Relax requirement of having a primary key declared on table

Simplify RowNumber Over limit dialect implementation

Deduplicate code in rownum limit emulation

Use the RSRC unique constraint traversal to determine if an order_by is stable

Deduplicate code in RSC, use the RSRC unique constraint traversal instead

Add an internal unique colset finder and relax complex $rs update/delete code

As long as there is something to correlate the subquery by we are good

Remove the "row-by-row shift" Ordered.pm pessimization

The idea this is necessary was born when working with buggy old 1.1x
SQLite versions back in 2008. This no longer seems to be necessary,
but if I am wrong we can always bring it back with an actual
storage capability flag

Fix embarrassing leak triggered on calling new_related on an uninserted object

Reason is a typo-ed hash element in a weaken() call back in 68f3b0dd. Add a
bunch of extra tests to catch this, and also better validate the new_related
behavior.

Fix rogue child disconnecting a parent $dbh - no idea how I even missed that

Majorly cleanup $rs->update/delete (no $rs-aware code should be in ::Storages)

Overhaul of variable \[] handling in populate, and a lot of extra tests

Now things like multi-value \[ ' ? || ?', [...], [...] ] and populate
datasets where the values of the \[] change per row work correctly

Switch from using execute_array to execute_for_fetch directly

This saves us from pivoting our data into column slices, which execute_array
promptly turns back into our initial row-based format to feed to
execute_for_fetch. Apart from the obvious speed gain, this saves a lot of
memory since it avoids 2 copies of the (possibly rather large) dataset

Fix ASE bulk_insert for bind changes in 0e773352

When using the bulk API, update the binds to the structure expected by
_execute_array. Also temporarily disable $sth->finish in the bulk API
codepath because for some reason it rolls everything back. Modify the
test to always exercise both codepaths from now on.

Factor out bindattr resolver and tighten code a bit

Zero functional changes

Allow users to disable the "DT in search" warning introduced in e1038213

Shuffle things around, maint/ is now a proper toolchain tooldir

Normalize -'s and _'s in maint names and Makefile options

remove stale link and doc readonly pool queries

transactions cause an implicit force to master, this documents how you can
override this to force to pool

Signed-off-by: Caleb Cushing <xenoterracide@gmail.com>

add a link to SQL::Abstract for WHERE clauses

Signed-off-by: Caleb Cushing <xenoterracide@gmail.com>

Fix forgotten whitespace - this is what I get for no prove xt

Only invoke the author Makefile.PL includes if we are not part of a `make` run

Comment typo

Forgotten diag and some clarifications of fc4b0448

Factor out the Makefile.PL into easily digestable snippets

Also add explicit guard for attempts to `make dist` from within non-author mode.
The rationale is to disallow dist building on machines that do not have the optdeps
installed (this still does not giarantee `make test` will run, but is at least a
nudge in the right direction)

Assert that complex bindtytyping passage works for the search() bind attribute

use more correct subsection links in POD

remove ::Storage::DBI::sth from POD as it should never have been public

fix PrettyPrint package link in POD

correct organization for SQLite POD

issue warning for DateTimes passed to ->search

Issue a carp_unique when DateTime objects are detected in binds in
::Storage::DBI with a pointer to the Cookbook recipe for formatting
DateTime objects for queries.

Add a test for the warning in t/inflate/datetime.t, which already has
TODO tests for DTs in ->search.

add POD on how to use Moose in custom ResultSets

Thanks to mst for providing the relevant info.

Trailing WS crusade - got to save them bits

Add link to SQLHackers documentation

- Fix typo (s/get_colum/get_column).

use Path::Class for finding subdirs of t and don't use global filehandle for slurping

refactored HashRefInflators internals

Rewrite the HRI bench to produce consistent numbers with less handholding

- Now we auto-pull from git what we want to benchmark (by default last
  2 commits + uncommitted changes if any)
- Run a very tight benchmarking loop, without requerying SQLite or have
  any other unrelated distractions
- use DumbBench (much saner implementation and results, though slower)

Add live Firebird test for paging fix 8b31f62e

Fix and test first_skip/skip_first limit dialects

Codebase didn't take in account the fact that limit bindvals for
SELECT FIRST x SKIP y and SELECT FIRST x SKIP y need to be inserted
at the head of the bind-assembly chain. Hence introducing a new
bind-chunk position 'pre_select'. While at it move the TOP dialect
to use it as well.

Extensive tests for both dialects, and also augment the test for
the Oracle RowNum dialect (no fixes necessary)

Standardize hobbs' email address

Release 0.08196

More test fixes following 4a425300 and new DBD::SQLite release

Fix some logic pitfalls in the require tracer

Fix the join/prefetch resolver when dealing with ''/undef/()

Modifying links to /condition to point to DBIC::Relationship::Base

Because that is where the condition description is to be found

Fix copyright... in November

Make both pg and mysql execute t/98savepoints.t if present

Make tests pass with DBIC_TRACE and DBIC_TRACE_PROFILE set

Fix TxnScopeGuard misbehaving on externally set $@ without inner exceptions

The guard is supposed to warn when it goes out of scope without a commit.
However this doesn't work if $@ was already set before the guard was created
and there were no eval{}s to clear it between definition and destruction.

Fixing this by storing a weakref to the current exception (if any) and
disregarding the contents of $@ if they match the ref we started with.

Abstract away the CORE::GLOBAL::require override code, foolproof tests

Rewrite prompted by some weirdness in Package::Stash::XS and require overrides
on older perls < 5.8.7. Besides the cleaner code add a kick-ass require
tracer.

More robust test suite skip-checks, check ENV before loading modules

Switch the nobindvars test from MySQL to the always available SQLite

Skip error/warn frames within CAG - saner callsite error messages this way

Documented $resultsource->name

Drop Math::BigInt optdep: no fixes concerning us were made since

Silence freetds/ODBC test warnings

The freetds+dynamic cursors TODO tests are throwing a warning for undef
in ->find, set the DBIC_NULLABLE_KEY_NOWARN env var to silence them.
This is not a solution of course, but we will deal with it when we
resolve the underlying issues with freetds and dynamic cursors.

Start caching the result of various bind_attribute_by_data_type invocations

Not only is this a speed win - it also avoids multiple querying of DBD
versions, which can lead to a memory leak (because version.pm is silly).

Still go even further and only check the VERSION of a DBD once unless the
DBD got reloaded.

Fix uninitializied warnings in ::Storage::Sybase::ASE, shuffle logic a bit

Fix incorrect Test::Builder dep (RT#72282)

Unhide ::SQLMaker::OracleJoins to properly appear on s.c.o.

Pass tests with DBICTEST_SQLITE_USE_FILE set

Changes for 135e8ce

Mark ->with_deferred_fk_checks as RDBMS-specific

Fixxor pause indexing

Ensure t/53lean_startup.t remains lean

Cleanup compose_namespace(), clarify leaktests wrt classdata

Fix segfault in t/storage/quote_names.t

Throw a sensible error when a malformed 'as'-spec is supplied

Cleanup warnings in t/55namespaces_cleaned, sophisticate require overload

Remove the transparrent hook lazy-pager-count experiment

It has proven a very stable and reliable implementation, but in the quest
for fatpacked DBIC should now go to the archives.

This atrocity will be removed from sqla, do not use here either

Fix find() pessimization when invoked with bare values

Add Changes entry for DBD::SQLite fixes.

Relax expected error message syntax.

SQLite 1.34 includes more detail in the error, allow any message
starting with "datatype mismatch".

Don't rely on GROUP BY and DISTINCT() sorting identically.

Release 0.08195

Fix oracle (I am so stupid)

Release 0.08194

fix link in replication introduction

Check DBD::Pg >= 2.17.2 with Pg >= 9.0 for BYTEA

Checks for DBD::Pg < 2.17.2 when using PostgreSQL version 9.0 or greater
when BYTEA column binds are encountered and prints a warning that a
newer version of DBD::Pg is needed for these types of columns to work in
bind_attribute_by_data_type in the Pg storage.

Skips the t/72pg_bytea.t tests for this combination of versions, as well
as for versions of DBD::Pg less than 2.9.2 regardless of Pg version.

fix popups during tests on Win32 (hopefully)

A tester reports that t/73oracle_hq.t causes an error popup on Win32.
This test was checking for DBD::Oracle (and thereby trying to load it)
before checking for the DBICTEST_ORA_DSN environment variables.

This commit moves the check for necessary environment variables for the
t/73oracle.t and t/73oracle_hq.t test files to before the module
requirements tests, thereby not trying to load DBD::Oracle, and causing
an error popup.

fix a couple test failures on Win32

Adds a skip_all to t/storage/global_destruction.t due to it causing a
segfault triggered by some sort of perl bug.

Adds a \Q...\E to t/94versioning.t in a warning regex as well as
changing a `perl -pi -e` to `perl -pi.bak -e` and removing an extraneous
semicolon for the test to pass.

Oracle: fix LOB conversions for non-LOBs (RT#69548)

Recalculates the LOB binds index for UPDATE queries, as it was
erroneosly not recalculated before. This resulted in non-LOB columns
being treated as LOBs by the SQL mangling code.

ribasushi also optimizes the code for the predominant case of queries
without LOBs in the WHERE clause.

Fix last test susceptible to spurious warnings, cleanup a bit

Reshuffle Makefile.PL a bit and add parallel-testing exception

Stop Data::Compare from loading plugins at will

Moar gitignore

Rip out forgotten dep

Really fix t/53lean_startup.t

fix $rs->populate([]) to be a no-op rather than an exception

Fix spurious test failures caused by use of Data::Compare

fix spurious PAUSE message

Release 0.08193

replaced Test::Deep::NoTest with Data::Compare to not break Test::Deep tests

Stop leaking cursors on oracle sequence retrieval