cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gerhard Froehlich" <g-froehl...@gmx.de>
Subject RE: [PATCH] Added Resource Monitor in ProgramGeneratorImpl
Date Tue, 11 Dec 2001 10:59:52 GMT
>-----Original Message-----
>From: Sylvain Wallez [mailto:sylvain.wallez@anyware-tech.com]
>Sent: Tuesday, December 11, 2001 11:53 AM
>To: cocoon-dev@xml.apache.org
>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: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org


Mime
View raw message