>-----Original Message----- >From: Sylvain Wallez [mailto:[EMAIL PROTECTED]] >Sent: Tuesday, December 11, 2001 11:53 AM >To: [EMAIL PROTECTED] >Subject: Re: [PATCH] Added Resource Monitor in ProgramGeneratorImpl > > > > >Gerhard Froehlich a écrit : > ><snip/> > >> > >> >Gerhard, >> > >> >After studying your patch, it appears to be broken : >> >ProgramGeneratorImpl is TreadSafe and thus a single instance is used for >> >*all* compiled components. So the lastModified date stored by >> >propertyChanged() is the date of the last modified program file found by >> >the ActiveMonitor. >> >The effect is that the next program requested using load() is >> >recompiled, but it is very unlikely to be the one that was modified : an >> >unchanged program is recompiled and the changed one is not recompiled :( >> >> Ahh Sh**. There can be more then one sitemap. I will look at it. Maybe >> we should pause the voting process ;). > >ProgramGenerator is also used for XSPs. And there is certainly more than >one XSP !
Ok, then we should remove the Monitor for now and go on to with Berins RT last night. It seems to be that this issue needs some more thoughts. >The vote won't be stopped : you deserve to be a committer, but my vote >has a warning about tests ;) Yupp recognized ;) >Sylvain. -------------------------------------------------------------- By making things absolutely clear, people will become confused -------------------------------------------------------------- --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, email: [EMAIL PROTECTED]
