incorrect sql for limit + count (DM 1.0)
Reported by Gary Yngve | June 22nd, 2010 @ 11:55 PM | in 1.1.1
In DM 1.0, the expression Foo.all(:limit=>2).count
produces the query
SELECT COUNT(*) FROM foos LIMIT 2
which is incorrect, as the precedence of LIMIT applies to the one
row
returned by count.
This is a big deal with many pagination plugins.
Comments and changes to this ticket
-
Martin Gamsjaeger (snusnu) October 29th, 2010 @ 04:46 PM
- State changed from “new” to “confirmed”
- Milestone set to “1.1”
- Milestone order changed from “0” to “0”
-
Dan Kubb (dkubb) October 31st, 2010 @ 09:55 AM
(from [1f9b8bf0d97a2458b9721e5fa02a7f1f69fdcdf3]) Add failing spec for #count called on a limited collection
[#1334 state:confirmed] [#1203 state:confirmed] [#1416 state:confirmed] http://github.com/datamapper/dm-aggregates/commit/1f9b8bf0d97a2458b...
-
Dan Kubb (dkubb) October 31st, 2010 @ 10:16 AM
(from [0b2afc96e4917e868853f3c9ddcc5c8f31ba58e5]) Mark #count on a limited collection spec as pending
[#1334 state:confirmed] [#1203 state:confirmed] [#1416 state:confirmed] http://github.com/datamapper/dm-aggregates/commit/0b2afc96e4917e868...
-
Dan Kubb (dkubb) March 16th, 2011 @ 10:41 PM
- Milestone changed from “1.1” to “1.1.1”
- Milestone order changed from “6” to “0”
-
Piotr Solnica (solnic) May 17th, 2011 @ 02:16 AM
- Tag changed from “dm-aggregates count limit” to “dm-aggregates”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
People watching this ticket
Gary Yngve
Martin Gamsjaeger (snusnu)
Attachments
Tags
Referenced by
-
1416
find_by_sql in dm-ar-finders interacts badly with dm-aggregates count
[#1334 state:confirmed] [#1203
state:confirmed] [#1416 st...
-
1203
aggregate function count broken for queries with :limit
[#1334 state:confirmed] [#1203
state:confirmed] [#1416 st...
-
1334
incorrect sql for limit + count (DM 1.0)
[#1334 state:confirmed] [#1203
state:confirmed] [#1416 st...
-
1416
find_by_sql in dm-ar-finders interacts badly with dm-aggregates count
[#1334 state:confirmed] [#1203
state:confirmed] [#1416 st...
-
1334
incorrect sql for limit + count (DM 1.0)
[#1334 state:confirmed] [#1203
state:confirmed] [#1416 st...
-
1203
aggregate function count broken for queries with :limit
[#1334 state:confirmed] [#1203
state:confirmed] [#1416 st...