cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gerhard Froehlich" <>
Subject RE: [PATCH] Added Resource Monitor in ProgramGeneratorImpl
Date Tue, 11 Dec 2001 10:59:52 GMT
>-----Original Message-----
>From: Sylvain Wallez []
>Sent: Tuesday, December 11, 2001 11:53 AM
>Subject: Re: [PATCH] Added Resource Monitor in ProgramGeneratorImpl
>Gerhard Froehlich a écrit :
>> >
>> >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 ;)


By making things absolutely clear, people will become confused

To unsubscribe, e-mail:
For additional commands, email:

View raw message