>-----Original Message-----
>From: Sylvain Wallez [mailto:[EMAIL PROTECTED]]
>Sent: Tuesday, December 11, 2001 11:34 AM
>To: [EMAIL PROTECTED]
>Subject: Re: [PATCH] Added Resource Monitor in ProgramGeneratorImpl
>
>
>
>
>Gerhard Froehlich a écrit :
>>
>> Hi,
>> as promised I started to implement the Excalibur
>> ActiveMonitor in Cocoon to increase our performance.
>>
>> I started at the ProgramGeneratorImpl and this Patch
>> contains the first draft.
>> Every 10sec the ActiveMonitor checks if the Sitemap
>> has changed.
>>
>> Please review it and give me some feedback!
>>
>> Berin: How about a additional URIResource in the
>> monitor package?
>>
>> Cheers
>> Gerhard
>>
>
>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 ;).
Cheers
Gerhard
--------------------------------------------
Give me ambiguity or give me something else.
--------------------------------------------
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]
