Re: got an exception

[Alexander Larsson <alexl redhat com>]

On Fri, 2005-05-06 at 15:05 -0400, Colin Walters wrote:
> On Fri, 2005-05-06 at 15:33 +0200, Alexander Larsson wrote:
> > On Fri, 2005-05-06 at 13:29 +0200, Alexander Larsson wrote:
> > > When working on something else i noticed i got this in the server:
> > > 
> > > Exception in thread "Yarrr client Reaper" java.lang.NullPointerException
> > >         at org.gnome.yarrr.HibernateUtil.addChangedObject(HibernateUtil.java:163)
> > >         at org.gnome.yarrr.Monitor.queueSignalChanged(Monitor.java:56)
> > >         at org.gnome.yarrr.ReferencableObject.signalChanged(ReferencableObject.java:70)
> > >         at org.gnome.yarrr.ToplevelReferencableObject.clientInactive(ToplevelReferencableObject.java:124)
> > >         at org.gnome.yarrr.ClientPoll.detectedInactive(ClientPoll.java:140)
> > >         at org.gnome.yarrr.Yarrr$ClientReaper.reap(Yarrr.java:323)
> > >         at org.gnome.yarrr.Yarrr$ClientReaper.run(Yarrr.java:334)
> > > 
> > > Maybe we broke reaping of inactive clients? 
> > > 
> > > I'll look at it in a while unless someone else gets to it first.
> > 
> > Ah, this is due to the change to queue signal notifications in a
> > thread-local variable and emit them at transaction commit time. However,
> > the reaper thread has no such thread-local variable.
> 
> How about this patch?

I guess that should work. Lets try it and see.

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a bookish skateboarding barbarian haunted by an iconic dead American 
confidante She's a strong-willed renegade nun who don't take no shit from 
nobody. They fight crime!