Skip Menu |
 

This queue is for tickets about the DBIx-Class CPAN distribution.

Report information
The Basics
Id: 56257
Status: resolved
Priority: 0/
Queue: DBIx-Class

People
Owner: Nobody in particular
Requestors: mrjobson [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.08122



Subject: Postgres, select count + for update generates sql error
Download (untitled) / with headers
text/plain 748b
my $lop = $schema->resultset('Lop')->search( { data_checked => {'<' => \'( CURRENT_DATE - INTERVAL \'1 week\' )' }, }, { select => ['id'], order_by => { -asc => ['id'] }, rows => 1, for => 'update' }, ); print $lop; # i want it to do count and to generate eror SET search_path TO contrib, public; SELECT COUNT( * ) FROM lop me WHERE ( ( data_checked < ( CURRENT_DATE - INTERVAL '1 week' ) ) ) FOR UPDATE DBIx::Class::ResultSet::count(): DBI Exception: DBD::Pg::st execute failed: ERROR: SELECT FOR UPDATE/SHARE is not allowed with aggregate functions [for Statement "SELECT COUNT( * ) FROM lop me WHERE ( ( data_checked < ( CURRENT_DATE - INTERVAL '1 week' ) ) ) FOR UPDATE" ] SQLA - 1.63 DBIC - 0.08120
Download (untitled) / with headers
text/plain 927b
On Sat Apr 03 15:54:50 2010, mrjobson@gmail.com wrote: Show quoted text
> my $lop = $schema->resultset('Lop')->search( > { > data_checked => {'<' => \'( CURRENT_DATE - INTERVAL \'1 week\'
)' }, Show quoted text
> }, > { > select => ['id'], > order_by => { -asc => ['id'] }, > rows => 1, > for => 'update' > }, > ); > print $lop; # i want it to do count and to generate eror > > SET search_path TO contrib, public; > SELECT COUNT( * ) FROM lop me WHERE ( ( data_checked < ( CURRENT_DATE - > INTERVAL '1 week' ) ) ) FOR UPDATE > DBIx::Class::ResultSet::count(): DBI Exception: DBD::Pg::st execute > failed: ERROR: SELECT FOR UPDATE/SHARE is not allowed with aggregate > functions [for Statement "SELECT COUNT( * ) FROM lop me WHERE ( ( > data_checked < ( CURRENT_DATE - INTERVAL '1 week' ) ) ) FOR UPDATE" ] > > SQLA - 1.63 > DBIC - 0.08120
So dbic should just drop the for attribute on ->count? Just double-checking.
From: mrjobson [...] gmail.com
On Sat Apr 03 20:22:59 2010, RIBASUSHI wrote: Show quoted text
> On Sat Apr 03 15:54:50 2010, mrjobson@gmail.com wrote:
> > my $lop = $schema->resultset('Lop')->search( > > { > > data_checked => {'<' => \'( CURRENT_DATE - INTERVAL \'1 week\'
> )' },
> > }, > > { > > select => ['id'], > > order_by => { -asc => ['id'] }, > > rows => 1, > > for => 'update' > > }, > > ); > > print $lop; # i want it to do count and to generate eror > > > > SET search_path TO contrib, public; > > SELECT COUNT( * ) FROM lop me WHERE ( ( data_checked < ( CURRENT_DATE - > > INTERVAL '1 week' ) ) ) FOR UPDATE > > DBIx::Class::ResultSet::count(): DBI Exception: DBD::Pg::st execute > > failed: ERROR: SELECT FOR UPDATE/SHARE is not allowed with aggregate > > functions [for Statement "SELECT COUNT( * ) FROM lop me WHERE ( ( > > data_checked < ( CURRENT_DATE - INTERVAL '1 week' ) ) ) FOR UPDATE" ] > > > > SQLA - 1.63 > > DBIC - 0.08120
> > So dbic should just drop the for attribute on ->count? Just
double-checking. Yes, it should.
Download (untitled) / with headers
text/plain 1.2k
On Mon Apr 05 06:06:36 2010, mrjobson@gmail.com wrote: Show quoted text
> On Sat Apr 03 20:22:59 2010, RIBASUSHI wrote:
> > On Sat Apr 03 15:54:50 2010, mrjobson@gmail.com wrote:
> > > my $lop = $schema->resultset('Lop')->search( > > > { > > > data_checked => {'<' => \'( CURRENT_DATE - INTERVAL \'1 week\'
> > )' },
> > > }, > > > { > > > select => ['id'], > > > order_by => { -asc => ['id'] }, > > > rows => 1, > > > for => 'update' > > > }, > > > ); > > > print $lop; # i want it to do count and to generate eror > > > > > > SET search_path TO contrib, public; > > > SELECT COUNT( * ) FROM lop me WHERE ( ( data_checked < (
CURRENT_DATE - Show quoted text
> > > INTERVAL '1 week' ) ) ) FOR UPDATE > > > DBIx::Class::ResultSet::count(): DBI Exception: DBD::Pg::st execute > > > failed: ERROR: SELECT FOR UPDATE/SHARE is not allowed with aggregate > > > functions [for Statement "SELECT COUNT( * ) FROM lop me WHERE ( ( > > > data_checked < ( CURRENT_DATE - INTERVAL '1 week' ) ) ) FOR UPDATE" ] > > > > > > SQLA - 1.63 > > > DBIC - 0.08120
> > > > So dbic should just drop the for attribute on ->count? Just
> double-checking. > > Yes, it should.
Fixed in [1]. Release coming in a couple of weeks. [1] http://dev.catalystframework.org/svnweb/bast/revision?rev=9297


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.