portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Scott T. Weaver" <scotts-jetspeed-l...@binary-designs.net>
Subject Re: latest FileSystemPAM mods
Date Wed, 02 Feb 2005 14:56:21 GMT
That was if somehow a portlet app was being deployed and the app already 
existed, i.e. deploy somehow fired in favor of redeploy. I think it was 
there to support a very slim chance of this happening. I don't think you 
need to worry about porting it.

Randy Watler wrote:

> Scott,
>
> I am confused by this change to FileSystemPAM:
>
> @@ -387,6 +387,12 @@
> // save it to the registry
> log.info("Saving the portlet.xml in the registry...");
> + if(registry.namedPortletApplicationExists(app.getName()))
> + {
> + log.warn("Deployment has found an app with the application name, 
> "+app.getName()+
> + ". The existing app will be removed in favor of this one.");
> + undeploy(paWar);
> + }
> registry.registerPortletApplication(app);
> log.info("Committing registry changes...");
>
>
> I would like this logic to somehow be moved to the 
> DeployPortletAppEventListener instead of bailing out in the middle of 
> a registration process here by undeploying. Under what conditions were 
> you trying to undeploy here? I do not understand how this could even 
> happen unless there were some kind of race condition involved... what 
> am I missing?
>
> I was not able to port this change forward into my version of 
> FileSystemPAM. We need to resolve this issue if we need to preserve 
> this functionality.
>
> Randy
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org
>
>
>


-- 
"Great minds discuss ideas. Average minds discuss events. Small minds discuss people."  -
Admiral Hyman Rickover

*******************************************
*           Scott T. Weaver               *
*         <weaver@apache.org>             *
*     <http://www.einnovation.com>        *
* --------------------------------------  *
*   Apache Jetspeed Enterprise Portal     *
*     Apache Pluto Portlet Container      *
*                                         *
* OpenEdit, Website Content Management    *
*     <http://www.openedit.org>           *
*******************************************


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


Mime
View raw message