Please avoid top-posting. You know the problem:

A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

   My answer below.

On Wed, May 22, 2013 at 06:16:20PM -0600, Andrew Trusty <atru...@gatech.edu> 
wrote:
> I think you just answered how the bug I reported off-list occurred but I
> was asking about the patch from revision 4530 to 4531 (when the "Cannot
> unpickle" ValueError was first introduced) that fixed a bug relating to
> unpickling.
> 
> On Wed, May 22, 2013 at 6:05 PM, Oleg Broytman <p...@phdru.name> wrote:
> > On Wed, May 22, 2013 at 05:46:40PM -0600, Andrew Trusty <
> > atru...@gatech.edu> wrote:
> > > I was
> > > wondering what the bug was that was fixed in 1.2.3 that necessitated this
> > > change. The site only says "Fixed a bug in cache handling while
> > > unpickling." which I found ironic because it broke caching in my app. Was
> > > the bug a memory leak due to having multiple instances with the same ID
> > in
> > > memory or was there something else? I ask because my app never seemed to
> > > experience any adverse side effects from taking advantage of this "bug".
> >
> >    This patch, I think:
> >
> > http://sourceforge.net/mailarchive/forum.php?thread_name=E1SXzWF-0005Cg-9K%40webwareforpython.org&forum_name=sqlobject-cvs
> >    The last line was not changed but id was removed from the code, that
> > how the last line became broken.

   I think the idea is to prevent getting two rows with the same id via
SELECT and unpickling.

Oleg.
-- 
     Oleg Broytman            http://phdru.name/            p...@phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to