portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ate Douma <...@douma.nu>
Subject Re: cvs commit: jakarta-jetspeed-2/components/registry/src/java/META-INF ojb_repository.xml
Date Fri, 11 Jun 2004 21:02:37 GMT


Scott T Weaver wrote:

> We might want to find an alternative to this fix as we will want to
> preserve user preferences that are tied to PortletEntities across
> re-deployments.  Auto deleting entities means deleting the associated
> preferences.
Yes, you are correct. I didn't think of that yet.

Still, unconnected child data shouldn't normally be kept in the database.
To keep this data when doing a redeployment I think the PA itself should be preserved.
During redeployment the configurations then should be merged/cleaned/updated.
I would prefer two different tasks: an undeploy and a redeploy so that an undeploy can clean
up the database.

> 
> On Fri, 2004-06-11 at 15:30, ate@apache.org wrote:
> 
>>ate         2004/06/11 12:30:49
>>
>>  Modified:    components/registry/src/java/META-INF ojb_repository.xml
>>  Log:
>>  Enabled PortletEntity as collection of PortletDefintion with auto delete="true".
>>  With the new auto generation of non-existing portlet entities on first request
>>  these really need to be removed also otherwise an undeploy/deploy sequence
>>  to update a PA would fail if invalid entries would still exists.
>>  CVS: ----------------------------------------------------------------------
>>  CVS: PR:
>>  CVS:   If this change addresses a PR in the problem report tracking
>>  CVS:   database, then enter the PR number(s) here.
>>  CVS: Obtained from:
>>  CVS:   If this change has been taken from another system, such as NCSA,
>>  CVS:   then name the system in this line, otherwise delete it.
>>  CVS: Submitted by:
>>  CVS:   If this code has been contributed to Apache by someone else; i.e.,
>>  CVS:   they sent us a patch or a new module, then include their name/email
>>  CVS:   address here. If this is your work then delete this line.
>>  CVS: Reviewed by:
>>  CVS:   If we are doing pre-commit code reviews and someone else has
>>  CVS:   reviewed your changes, include their name(s) here.
>>  CVS:   If you have not had it reviewed then delete this line.
>>  
>>  Revision  Changes    Path
>>  1.6       +1 -4      jakarta-jetspeed-2/components/registry/src/java/META-INF/ojb_repository.xml
>>  
>>  Index: ojb_repository.xml
>>  ===================================================================
>>  RCS file: /home/cvs/jakarta-jetspeed-2/components/registry/src/java/META-INF/ojb_repository.xml,v
>>  retrieving revision 1.5
>>  retrieving revision 1.6
>>  diff -u -r1.5 -r1.6
>>  --- ojb_repository.xml	8 Jun 2004 01:32:53 -0000	1.5
>>  +++ ojb_repository.xml	11 Jun 2004 19:30:49 -0000	1.6
>>  @@ -429,8 +429,6 @@
>>   
>>         </collection-descriptor>
>>   
>>  -
>>  -   <!--      
>>         <collection-descriptor
>>            name="portletEntities"
>>            element-class-ref="org.apache.jetspeed.components.portletentity.PortletEntityImpl"
>>  @@ -438,10 +436,9 @@
>>         	  auto-update = "true"
>>         	  auto-retrieve = "true"
>>         >
>>  -    
>>              <inverse-foreignkey field-ref="portletId"/>
>>        </collection-descriptor>
>>  -       -->
>>  +
>>      </class-descriptor>
>>      
>>   
>>  
>>  
>>  
>>
>>---------------------------------------------------------------------
>>To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
>>For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.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