only add rels once when seen repeatedly

first tests for identifier remapping and a slight refactoring

reorganise dq expr tests slightly

move join inference cleverness into a role

need to clear converter after setting limit dialect

first (naive) cut at identifier remapping and join injection

switch to delete+local from clear_* methods

STRAIGHT_JOIN support for MySQL

literals on the other side of = are acceptable for a fixed condition

fix foreign key identification in SQLT parser

Fix lean startup for Data::Compare -> Data::Dumper::Concise in ResultSet

fix unresolvable condition handling

fix test to use qw as (so far as I can see) originally intended

fix condition extraction for new_result

maintain condition order (left versus right) in _stack_cond

_resolve_condition for DQ exprs now provides correct value_meta

factor out _extract_top_level_conditions and use it to rewrite _stack_cond

let rowparser introspect DQ join conds

DBICTest result class now loading ExprDeclare

factor out join cond to hashref code, return unresolvable conditions correctly

make extract_fixed_values work without an alias, make _extract_fixed_condition_columns use it

move _extract_fixed dq methods into resultsource to share between rs and row

some support for dq in rs condition merging and populate

move reverse_relationship_info "is join a hashref" check into __strip_relcond

support expr {} in join cond

allow DQ expr {} blocks as first arg to search

Merge branch 'master' into dq2eb

This is an expensive author-only test, move to xt/

Also get xt tests first in line (won't matter much with -s)

Make sure DBICTest is always loaded first (purely bookkeep)

Work around TB1.5 hanging with 5.19 - use unreleased github patch

Revert 2c300e45 - new backwards compatible SQLT got released

Work is being done to reintroduce the feature with a softer push:
https://github.com/dbsrgits/sql-translator/pull/26#issuecomment-27644756

Clarify what happens on distinct + get_column($aggregate_alias)

Centralize and sanify generation of synthetic group_by criteria

This should solve multiple deficiencies when a limited prefetch is ordered
by non-local columns (especially pre-multiplied ones). Also intelligently
solves many problems combining distinct with various non-selecting external
order criteria.

Most current behaviors should remain unaffected except for cases of
blatantly incorrect query generation (fingercross)

fix typo in FATAL ERROR message.

Move database-independent mysql SQLMaker test

set nulls => "none" in order nodes under GenericSubquery

fixup INNER use for mysql 3

identifier parsing to make mysql subselect wrapping work within literals

remove dead code from _extract_order_criteria

_extract_order_criteria can now handle ident only or full criteria

t/prefetch/o2m_o2m_order_by_with_limit.t: literal identifier now gets quoted

move select arg handling to _select_to_dq to enable creating $rs->_as_select_dq

_extract_order_criteria should return column names, not full criteria

use more DQ in DBIHacks to avoid literals everywhere

t/sqlmaker/limit_dialects/generic_subq.t: unselected column given gensym alias

render order by elements without quoting to get identifier names in _extract_order_criteria

THROW AWAY this commit on rebase - dirty bootstrap

Simplify DBIHacks signatures

The cross-polination of $attrs and various subparts is becoming a nightmare
to maintain. Straighten up most ::DBIHacks to pass stuff around as a single
$attrs bag (no functional changes, this is a temp-stage cleanup)

The idea is to prepare for an rsattrs object (later on all we will need to
do is replace the hash-grabs with methods, but this requires much more
thought and design, small steps)

Affected signatures in :DBIHacks:
_prune_unused_joins()
_resolve_aliastypes_from_select_args()
_adjust_select_args_for_complex_prefetch()
_extract_fixed_condition_columns()

Not touching to facilitate easier DQ merge (TODO in a sense):
_order_by_is_stable()
_main_source_order_by_portion_is_stable()

Temporarily freeze travis SQLT on 0.11016, until the SQLT debacle is resolved

https://github.com/dbsrgits/sql-translator/pull/26#issuecomment-27472588

Test more JSON::Any backends, and reorder the selection in terms of desirability

DWIW is the most lax JSON parser and thus is always preferred for CLI work

Work around JSON::Any not being future-proof

https://rt.cpan.org/Ticket/Display.html?id=89853
https://github.com/perigrin/json-any/pull/2

Consistently use "eq ''" when checking for exception refness

Explicitly stringify when checking for existing exception

This fixes errors with exception objects with incomplete
overloading (like Log::Report::Dispatcher::Try).

Fix a47f273b change to be in line with the Row->Result rename of fb13a49f

fix some pod links

Better render of $POSTMORTEM

Add a MAKEFLAGS parallel execution config on travis

Get TAP::Harness to time tests travis-wide

Remove anything preinstalled on the travis perls when CLEANTEST is true

This will give us yet another test point

Smoother concurrent test-flow

The noop trick initializes the STDOUT, so that the TAP::Harness
issued IO::Select->can_read calls (which are blocking wtf wtf wtf)
keep spinning and scheduling jobs

This results in an overall much smoother job-queue drainage, since
the Harness blocks less

(ideally this needs to be addressed in T::H, but a quick patchjob
broke everything so tabling it for now)

Build more recent perls concurrently

Check that 64 bit in-SQLite math works regardless of ivsize

Expand on top of 04ab4eb1 and 579d1966, work inspired by charsbar's
https://github.com/charsbar/dbd-sqlite-authortests

Tested by passing the following on an old and recent DBI and a mix of ivsizes

for p in 5.8.1 5.8.2 5.8.3 5.8.5 5.18.0 5.18.1; do
  perlbrew switch $p
  for v in 1.29 1.31 1.33 1.35 1.37 1.39 1.40 ; do
    PERL_CPANM_OPT= cpanm -qn DBD::SQLite@$v && echo -n "$p " && perl -V:ivsize && prove -l t/752sqlite.t || break
  done
done

@mattp- now *that's* attention to detail :)

Fix exceedingly sloppy SQLite warning workaround from 04ab4eb1

The previous version managed not only to *not* silence any warning but also
break the test checking for lack of said warnings. In addition the silencing
regex was unacceptably broad, possibly missing some important misbinds.

riba pull your shit together man
a.k.a. this is what happens when there are no code reviewers :(((

Forgotten remnant of 655d80dd (void by f207111d)

Really work around https://github.com/travis-ci/travis-ci/issues/1477

Just upgrade cpanm by hand with --dev

Add more autogenerated test exceptions, in fact do a carpetbomb

This is necessary at least for a while until the war on the user passes

Streamline postmortem analysis, and fix non-essential-test report being lost

It got eaten by lack of STDERR output after fixage from a4c52abc, GAH!

improve order by error message

FetchFirst does not inject an order because me.title is unique

t/sqlmaker/limit_dialects/toplimit.t - add missing me.id - fairly sure this was a bug

complete murdering emulate_limit

t/sqlmaker/quotes/quotes_newstyle.t - ORDER BY parsing means we quote, but not misquote

t/sqlmaker/quotes/quotes.t - ORDER BY parsing means we quote, but not misquote

clean up OracleJoins

oraclejoins fix

t/sqlmaker/oracle.t - parens and quoting fixes

t/sqlmaker/limit_dialects/toplimit.t - ORDER BY title -> me.title

t/sqlmaker/limit_dialects/rownum.t - ROWNUM -> ROWNUM AS

t/sqlmaker/limit_dialects/rno.t - ORDER BY on column of "main" table

t/sqlmaker/limit_dialects/fetch_first.t - minor SQL tweaks

Add a 9m soft / 9.5m hard timeout option to bin/cpan runs

This makes sure we do not abort a travis run without meaningful output

Reenable 5.10 travis smokes

https://rt.cpan.org/Ticket/Display.html?id=88903 got solved \o/

Correct attribution of nonessential skipped tests origin

MULTICREATE_DEBUG + TRACE_PROFILE=console_monochrome results in >4MB log

Pod/comment fixes

Make the dbh error handler installer `Perl -d` friendly

GC in perl is a mess sometimes - a literal sub { ... } within a try { ... } goes
wonky, but the equivalent coderef stored in a scalar is fine >.<

Move classdata handling to DBICTest::Util::LeakTracer

Silence nonessential test warning (last outstanding \o/)

correctly carry DESC on simple orderings for complex prefetch

make multicolumn IN work for update/delete

switch _adjust_select_args_for_complex_prefetch order by handling to part-DQ

switch having code in _count_subq_rs to DQ SQLA, update regexp

make _resolve_aliastypes_from_select_args handle an undef select

clear renderer objects before dicking around with them as well as after

Many improvements of bigint handling across various DBD::SQLite versions

This started as a patch by ilmari to bind all integers as SQL_BIGINT
as opposed to SQL_INTEGER. While this change itself worked fine, further
testing revealed we never actually bound 'bigint' column types in the
first place. When this was rectified all hell broke loose.

The current situation per DBD::SQLite version

1.29 ~ 1.33:  everything works fine, for large values DBD::SQLite loses
              precision on perl binaries with ivsize == 4 (just like integer
              math within perl itself)

1.35:         Nothing works *regardless* of ivsize. Binding a >32 bit value
              as either SQL_INTEGER or as SQL_BIGINT is impossible (an
              exception is thrown from within DBD::SQLite). DBIC simply strips
              the bindtype altogether and issues an "UPGRADE!" warning.

1.37 ~ 1.40:  Everything works, except for the annoying fact that DBD::SQLite
              issues a spurious 'datatype mismatch' warning when one tries
              to bind a >32 bit value with ivsize == 4. RT#76395
              Warning is silenced for the time being within the guts of DBIC.

On the SQL_INTEGER => SQL_BIGINT change: according to
http://www.sqlite.org/datatype3.html#storageclasses all numeric types are
dynamically allocated up to 8 bytes per individual value. Thus it should be
safe and non-wasteful to bind everything as SQL_BIGINT and have SQLite deal
with storage/comparisons however it deems correct.

Add explicit notes wrt DBI/DBD::SQLite dep versions

Make $SIG{__WARN__} overrides more Carp::Always friendly

Warn about non-integer values for all integer bind types, not just SQL_INTEGER

Use explicit variable on loop

Fix and guard against erroneous use of list context in internal DBIC code

This situation is dangerous in case the end-user employs something like
DBIx::Class::Helper::ResultSet::IgnoreWantarray

Besides the trivial fix in ::Row::copy, this commit introduces the
DBIC_ASSERT_NO_INTERNAL_WANTARRAY=1 mechanism (which will be
automatically picked up and engaged by our CI due to eed5492f). As a
bonus the check is usable on DBIC dependencies as well.