Stop needlesly invoking the $rs bool overload

Reorder prefetch logic for optimal execution (no functional changes)

Avoid yet another expensive try call

Normalize varnames (zero functional changes)

Fix test description (DrHyde++)

Merge branch 'master' into topic/constructor_rewrite

Add some extra code to enforce the assumption that any bind type constant
is accessible in _dbi_attrs_for_bind, or in other words that all necessary
DBDs are already loaded (concept originally introduced in ad7c50fc)

Without this the combination of 9930caaf7e (do not recalculate bind attrs
on dbh_do retry) and a2f228547 (do not wrap iterators in dbh_do) can result
in _dbi_attrs_for_bind being called before DBI/DBD::* has been loaded at all

Remove idiotic RowCountOrGenericSubQ - it will never work as part of as_query

I am not sure how I overlooked this, blame is on me :(

Replacing with a ROWCOUNT fallback at the appropriate place

Changes pod references to deprecated SQLT methods

quote_table_names/quote_field_names is deprecated, refer to quote_identifiers
instead.

Add a TODO to catch when Text::Balanced RT#74994 is finally fixed

In example code, placing a necessary semicolon after a try/catch code-block, and correcting a nearby variable name.

Fix oversight in subqueried MySQL update/delete

Release v0.08210

Trap and display STDERR during travis test runs

More shuffling of travis testdeps

Make sure external DBIC envvars do not cause tests to fail

Run the appropriate tests on travis to detect future problems

Fix incorrect warning/exception originator

Until now we would report the second to last function as the
exception originator, whereas we need to report the first after
skipping the usual suspects of throw_exception etc

Do not repeatedly trace test schema populate under TravisCI

Extra TODO tests for a txn_guard silencing problem

Work around SQLite's RT#79576

Includes a warning that data may have been corrupted in previous
DBIC versions, with resources on how to deal with this.

Saner check for non-integer values bound as integers in SQLite

Simplify _select_args_to_query - contortions not needed since 0e773352

Refactor sth preparation/binding - no functional changes

Notable internal changes:

- _dbi_attrs_for_bind call is moved out of _dbh_execute - there is no point
  recalculating these on retry
- _dbh_execute changed signature: $ident => $bind_attrs
- sth, _sth and _dbh_sth are no more - instead we now have _prepare_sth and
  _bind_sth_params

The test in t/storage/base.t has not been actually working for years (it
did not register a spurious success with $count == 0). Removing it is safe
as t/storage/reconnect.t tests the same codepath more thoroughly

DESCRIPTION typo fix

Fix test warnings left in by 11343b34

improve outline by introducing synopsis subheadings

improve mark-up

development info is less important than user info

Add daxim to contribs, specify main doc encoding

9 years of perl and I somehow did not know that...

Fix copypasta - strictures test needs no global lock

Only display output of failed workers in parallel_installdeps_notest

File::Path needs to be upgraded separately on smoking

Add deprecation warnings for cols/include_columns

* Also update docs and include test

Fix test failing with some tracing envvars (and smoke on travis too)

EXPERIMENTAL Release v0.08242

Properly wrap a subquery around grouped resultsets in case of $rscol->func($func)

Radically rethink complex prefetch - make most useful cases just work (tm)

TL;DR: mst - I AM SORRY!!! I will rebase the dq branch for you when this
pile of eyebleed goes stable.

The long version - since we now allow arbitrary prefetch, the old
_prefetch_selector_range mechanism doesn't cut it anymore. Instead we
recognize prefetch solely based on _related_results_construction.
Furthermore group_by/limits do not play well with right-side order_by
(which we now also support, by transforming foreign order criteria into
aggregates).

Thus a much more powerful introspection is needed to decide what goes on
the inside and outside of the prefetch subquery. This is mostly done now
by the augmented _resolve_aliastypes_from_select_args to track
identifiers it saw (97e130fa48), and by extra logic considering what
exactly are we grouping by.

Everything is done while observing the "group over selection +
aggregates only" rule, which sould allow us to remain RDBMS agnostic
(even for pathological cases of "MySQL-ish aggregates").

As a bonus more cases of "the user knows what they are doing" are now
correctly recognized and left alone. See a t/prefetch/with_limit.t diff
for a general idea of the scope of improvements.

Yes - there is more regexing crap in the codebase now, and it is
possible we will call _resolve_aliastypes_from_select_args up to 4(!!!)
times per statement preparation. However this allows us to establish a
set of test cases towards which to write optimizations/flog the dq
framework.

There is no point of ordering the insides of a complex prefetch without a limit

Consider unselected order_by during complex subqueried prefetch

Augment _resolve_aliastypes_from_select_args to collect the column names
it sees, allowing it to replace _extract_condition_columns() entirely.

In the process fix a number of *incorrect* limit_dialect tests

Warn in case of iterative collapse being upgraded to an eager cursor slurp

Move CDBI code out of the main ResultSet implementation

Still do null-branch pruning when we are using our own inflate_result()

52864fbd was too aggressive in removing stuff - inheritance no longer
matters, but which method we will be calling does. If we are still using
our own inflate_result() there is no point of doing the double work of
blessing and then skipping null branches.

Forgotten cruft after 52864fbd5

Rename some private methods/attributes to align with what they do

No functional changes, just a s///

Remove some superfluous expensive try blocks

We already know that the relationship exists, and ->related_source without
a schema returns undef anyway

Simplify more - remove loop and cleanup the while cond

An underdefined root has to use the %cur_row_ids container

Otherwise we are polluting $cur_row_data with the extra "virtual id column"
value, and may end up stashing it back on iterative cursor parsing

MOAR optimization - tracking is_new_res is double work

We can get the root node assembler to do that for us instead

Also remove more of the superfluous ||= assignments

BAIL_OUT stops the entire test run - do not want that

Scale back validation of the 'as' attribute (revert 95e41036)

It turned out that users employ invalid-but-sensical inflation maps in
production. Remove the early sanity check of the infmap and move the code
throwing an exception all the way back to ::Row

Follow through this thread for more details:
http://lists.scsys.co.uk/pipermail/dbix-class/2013-February/011115.html

Clarify result_class tweaking on active/cached cursors

We need this to simplify logic of result_class type caching

Remove test cruft forgotten after 52864fbd

Do not rely on Data::Dumper to produce rolled out hashrefs

Merge branch 'master' into topic/constructor_rewrite

Abraxxa-proof the distbuild (disallow `make manifest dist`)

This does not work under dmake (there is no $(MAKECMDGOALS)), but it does
not seem to have adverse effects either

Exclude DBICTest namespace from stacktraces

Augment 1b6cbedd (reverting my moronic use of preamble in EU::MM)

Instead of whacking things during the MY::distdir override, properly
override MY::postamble for upload prereq check injection

move the location of the extra upload checks so it isn't the default target

This should result in equivalent behavior since it is still before the
main upload rule.

More comprehensive test of qualifier stripping for update/delete

Change the rt link from one requiring a login to a publicly accessible one

Also switch to https:// whatever can be reasonably switched

Reorganize and freshen up the DocMap

It accumulated lots of stuff that is confusing / not relevant for beginners

Move DocMap link to top of DBIx::Class to prevent it being missed

Reorganize the section to be a bit more comprehensive

Remove outdated (and kinda offensive) CONTRIBUTING text

A replacement should come shortly, courtesy of Getty

And more fulltest dep conflicts

Clarify wtf do we have a _bool

Require latest EU::MM at dist-building time to avoid $-escaping issues

Do not auto-install dist-building dependencies in author mode

Instead only require them for actual make dist/distdir/upload runs

Smoke all branches on travis except for blocked* and wip/*

Another adjustment of travis depinstall order

Punt TODO until the dateops work merges

This infrastructure was removed ages ago by 263e41be

Remove TODO that could never be made to work (revert c49fcf72)

I do not know what groditi was thinking - the behavior codified makes 0 sense

Release v0.08209

Fulltests deps clarification

- CDBI::Plugin::DeepAbstractSearch already depends on CDBI
- Install DateTime earlier in the CLEANTEST=false cycle

Guard against exceptions in _describe_connection

Stuff like this happens otherwise:
http://lists.scsys.co.uk/pipermail/dbix-class/2013-January/011085.html

Whitespace-check the travis scripts as well

Shuffle travis CLEANTEST=false depchain again

Test::Strict is smarter now

Regex /o is *EVIL* - no idea why I even thought of using that

Warn on signs of Moose-borkage (in case it gets to be the first new())

Adjust CLEANTEST=false depinstall a bit more

Fix cookbook - missing cond in 'Equivalent SQL'

Instrument the lockfile creation failure

Attempting to get to the bottom of http://www.cpantesters.org/cpan/report/27f16417-89ea-1014-a33e-b361b1e9163f

Fix regression breaking search on prefetched rel (broken by 5e2a0518)

Boy 0.08205 was a bad release, so much crap went in undetected by
any of our tests :( Maybe it's time for Devel::Cover...?

Uncomment disabled test (not sure why it was)

EXPERIMENTAL Release v0.08241

Clarify prefetch documentation a bit, remove the nonsensical 0.09 warning

Sanity-check columns added as primary key

Optimize the HRI-direct collapser even more

Since we know that we will never output null-branches in the case of
HRI, we can wrap the related assemblers in a definedness condition to
never fire on a null branch, and skip entirely the calculation of
non-NULL %cur_row_ids.

Fatalize warnings within the compiled row parsers

Encountering an undef value anywhere in this code is a sign of some glaring
omission. Perhaps a nullable column is annotated as is_nullable => 0 or
something similar. When this is the case we need the collapsing to stop
right there, as it is no longer safe to proceed.

Modify the null-branch pruning introduced in ce556881, restore compat

In the case of "direct to HRI" we want to throw stuff away as before. In
other cases however we no longer take an educated guess when to prune.
Instead we mark each null-branch by simply blessing its containing
arrayref. This allows us to keep the arguments to inflate_result 100%
backwards compatible, while at the same time allowing "enlightened"
inflate_result implementations to skip over the dead branches with
minimal effort.

Both ::Row::inflate_result and ::HRI::inflate_result were adjusted to
react correctly to these marks. As a result the HRI folder gained
another 5% speedup (unless sidestepped by the direct-to-HRI code, which
is naturally much much more efficient).

The current name of the "null class" is stored in the global
$DBIx::Class::ResultSource::RowParser::Util::null_branch_class. Given
that the entire codebase begs for standalone CPAN extraction this should
do for the time being.

While implementing the test changes it became clear that
Test::More::is_deeply does not cut it (it ignores the blessed-ness of
structures by design: https://github.com/schwern/test-more/issues/347).
As a result a dep on Test::Deep was added, and several of the tests
looking at inflate_result RVs were converted.

Add optdep on CPAN::Uploader to get an effortless `make upload`

Even more distbuilding checks

Frew's environment (and *only* his environment) keeps producing weirdly
broken dists. Add an extra check for this until we figure out how exactly
his environment is broken.

Extra check tested on cygwin and strawberry for good measure.

Release v0.08208

Add optdep on CPAN::Uploader to get an effortless `make upload`

Even more distbuilding checks

Frew's environment (and *only* his environment) keeps producing weirdly
broken dists. Add an extra check for this until we figure out how exactly
his environment is broken.

Extra check tested on cygwin and strawberry for good measure.

New Test::Strict no longer detects Moose::Role (RT#83433) - workaround

Readjust the parallel no-test dependency install order

Better utilize the massive amount of CPUs we have available

It appears that travis drastically increased the amount of available CPUs

Well, this is awesome - we just happen to be equipped to take advantage
of this ;)

Remove the branch-name limitations - now a smoke takes fraction of the time
it used to, no point to limit what we smoke anymore.

At the same time reduce the amount of threads building the perls - older
binaries do not build very well on -j32. This is a temporary thing - the
perl-building will soon be exported away anyway.

New Test::Strict no longer detects Moose::Role (RT#83433) - workaround

Prevent SQL::Translator::Producer::YAML from seeing our potential $dbh

The failure mode here is extremely convoluted, full of unimaginable fail
and makes baby jesus cry :(

The long story: there are two ways to supply a $schema to the SQLT producer:
via an argument to translate() (sane) or at the time of translator instance
creation (less sane). In the second case this becomes a problem because some
serializing producers (e.g. SQLT::Producer::YAML) take the *entire*
SQLT instance and dump it to YAML. This includes the arguments which include
our schema with its storage and potentially its $dbh. This causes the
weird warnings described in e.g. RT#75394, because YAML creates a dead
DBI object upon de-serialization of said arguments. What's even more sad is
that while the YAML Producer dumps the entire SQLT instance, the YAML Parser
simply takes the ->{schema} part of this structure and throws everything
else away.

Of course we can not just fix the ::Producer - folks may very well be relying
on the structure it spits out (nobody knows if the *only* consumer of the
Producer::YAML output is in fact Parser::YAML). Really the only thing we can
know (well not know, but reasonably assume) is that there is no way that any
user relies on the accessing the storage passed in with a schema instance to
be parsed.

So what we do is the least invasive thing of all - at translate() time we
check the state of the passed in $schema and if it does have a storage
instance we simply detach it (by cloning the $schema instance) and as a
result everything remains happy.