make Ordered support skimming (eg. TOP) limits historic_rejected/force_ordered_order_by
authorRafael Kitover <rkitover@cpan.org>
Mon, 12 Mar 2012 16:45:45 +0000 (12:45 -0400)
committerRafael Kitover <rkitover@cpan.org>
Mon, 12 Mar 2012 16:45:45 +0000 (12:45 -0400)
commit37beb70f841027368caa2121c08106148cdee78c
treee90ce80f2979cd5eb992bbf01e58396a684d6430
parent48775dd11096e6a57bde9532f47412c90d87975c
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.
lib/DBIx/Class/Ordered.pm