Skip Menu |
 

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

Report information
The Basics
Id: 5324
Status: resolved
Priority: 0/
Queue: DBIx-SearchBuilder

People
Owner: Nobody in particular
Requestors: rg [...] progtech.net
Cc:
AdminCc:

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

Attachments
DBIx-SearchBuilder.patch



Return-Path: <j [...] fsck.com>
Delivered-To: cpan-bug+dbix-searchbuilder [...] pallas.eruditorum.org
Received: by pallas.eruditorum.org (Postfix, from userid 1037) id 8D6C1114EF; Fri, 13 Feb 2004 15:51:30 -0500 (EST)
Received: from ptserver.progtech.net (pD9E8A3D4.dip.t-dialin.net [217.232.163.212]) by pallas.eruditorum.org (Postfix) with ESMTP id A5B08114FB for <jesse [...] bestpractical.com>; Fri, 13 Feb 2004 14:30:55 -0500 (EST)
Received: from PROGTECH.net (isis.muc.progtech.intern [10.25.0.100]) by ptserver.progtech.net (8.12.9p2/8.12.9) with ESMTP id i1DJUqim031131 for <jesse [...] bestpractical.com>; Fri, 13 Feb 2004 20:30:53 +0100 (CET) (envelope-from rg [...] PROGTECH.net)
Message-Id: <402D25EC.2090607 [...] PROGTECH.net>
Date: Fri, 13 Feb 2004 20:30:52 +0100
From: Rolf Grossmann <rg [...] progtech.net>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.5b) Gecko/20030905
X-Accept-Language: en,German [de]
MIME-Version: 1.0
To: Jesse Vincent <jesse [...] bestpractical.com>
Subject: Bug in DBIx::SearchBuilder
References: <C1323ACC-5E4C-11D8-8E7B-000A95AA631E [...] bestpractical.com>
In-Reply-To: <C1323ACC-5E4C-11D8-8E7B-000A95AA631E [...] bestpractical.com>
Content-Type: multipart/mixed; boundary="------------050801060306090005020904"
X-Spam-Score: 0 ()
X-Scanned-BY: MIMEDefang 2.38
Resent-From: j [...] fsck.com
Resent-Date: Fri, 13 Feb 2004 15:51:30 -0500
Resent-To: bug-dbix-searchbuilder [...] rt.cpan.org
Resent-Message-Id: <20040213205130.8D6C1114EF [...] pallas.eruditorum.org>
Content-Length: 0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-RT-Original-Encoding: us-ascii
Content-Length: 853
Download (untitled) / with headers
text/plain 853b
Hi Jesse, Jesse Vincent wrote: Show quoted text
> RT 3.0.9 requires DBIx::SearchBuilder 0.97 which was just released to > CPAN. This version is identical to the 0.97_02 development release.
There's a bug in DBIx::SearchBuilder that I've been meaning to tell you (but forgot, sorry). Remembered from this mail and it's still present. Goes as this: The _gen_alternate_cache_key function does not handle values that include a operator (and are therefore perl HASHes). As a result, the generated key contains the hashes memory address instead of the value, which for me resulted in some funny non-deterministic behaviour when adresses for different objects were hit. Possible patch is attached (and I hope that will make things clearer than my description). I haven't checked if that is the only time the value is used like that, it did fix my problem. Rolf
Content-Type: text/plain; name="DBIx-SearchBuilder.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline; filename="DBIx-SearchBuilder.patch"
X-RT-Original-Encoding: iso-8859-1
Content-Length: 541
--- /usr/ports/databases/p5-DBIx-SearchBuilder/work/DBIx-SearchBuilder-0.96/SearchBuilder/Record/Cachable.pm Fri Jan 2 22:55:29 2004 +++ /usr/dist/local/lib/perl5/site_perl/5.8.2/DBIx/SearchBuilder/Record/Cachable.pm Wed Jan 28 20:39:01 2004 @@ -257,6 +257,7 @@ my ($this, %attr) = @_; my $cache_key=$this->Table() . ':'; while (my ($key, $value) = each %attr) { + if (ref($value) eq "HASH") { $value = $value->{value}; } $cache_key .= ($key ||'__undef'). '=' . ($value||'__undef') . ','; } chop ($cache_key);
Content-Type: text/plain
Content-Disposition: inline
Content-Transfer-Encoding: binary
MIME-Version: 1.0
X-Mailer: MIME-tools 5.405 (Entity 5.404)
From: stephen [...] jadevine.org.uk
X-RT-Original-Encoding: iso-8859-1
Content-Length: 310
Download (untitled) / with headers
text/plain 310b
This patch makes a huge difference to me, with logging on i get 350 lines of logged queries pre-patch and 155 post-patch for displaying the same ticket. This is on Debian unstable, postgres 7.4.1, perl 5.8.3, rt 3.0.9 I hope this makes it into a release of DBIx::SBr sometime soon. Thanks, Stephen Quinney
Content-Type: text/plain
Content-Disposition: inline
Content-Transfer-Encoding: binary
MIME-Version: 1.0
X-Mailer: MIME-tools 5.417 (Entity 5.417)
X-RT-Original-Encoding: iso-8859-1
Content-Length: 426
Download (untitled) / with headers
text/plain 426b
[guest - Fri Feb 20 11:44:43 2004]: Show quoted text
> This patch makes a huge difference to me, with logging on i get 350 > lines of logged queries pre-patch and 155 post-patch for displaying the > same ticket. > > This is on Debian unstable, postgres 7.4.1, perl 5.8.3, rt 3.0.9 > > I hope this makes it into a release of DBIx::SBr sometime soon. >
Appears to have been applied forever ago. resolving. Show quoted text
> Thanks, > > Stephen Quinney


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.