portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephan Hesmer" <shes...@raleigh.ibm.com>
Subject Re: Synchronized and PortletConfig
Date Thu, 01 Mar 2001 13:15:34 GMT

Santiago Gala wrote:

> I agree in whole. This work is needed. We have been delaying this effort,
due to the new
> PortletAPI, but I think the changes go in the right direction.
>
> I tested having PortletSets cached in the session. This brought the
problem that portlets
> were not being correctly expired during the session, as expiration happens
in getPortlet()
> during creation of the PortletSet.
>
> I suggest taking out setPortletConfig() from the Portlet API and changing
to init(config),
>  so that there is no reasonable way to spoil the config again.
>
>
> We could do the following:
>
> - Maybe you could want to have commit rights, or will the changes be
commited by Ingo?
> - You post the changes' description for discussion, but I think the best
communication
> would be in the following points...
> - take 24-48 hours for people to commit other "minor" changes in the main
branch
> (I will commit the JAXP/TRAX changes, don't think they interfere)
> - open a branch for this work.
> - apply your patches in this branch
> - test a little bit
> - have a IRC meeting to discuss about any pending problems or unforessen
implications
> (maybe we understand things in different ways, and this can lead to
divergent work).
> Also, if the main developers understand the changes,and the reasons, it is
less
> likely that incompatible changes that spoil the model appear later.
> - test more :) (don't forget the jsp classes, that don't get compiled
during ./build)
> - merge the patches into the main branch
>
>
> The reasons is that I have also some changes in related classes, and while
I don't think I have problems,
> maybe I have seen things that you forgot, or the other way round. I prefer
having a branch, so that we
> can take time to think if something breaks (conflicts) without being
pressed by Sam Ruby's GUMP :)
>

I would suggest not to apply the changes in the current jetspeed, but that I
implement the changes directly with the new Portlet API. I will implement
the new API for Jetspeed here at IBM.

Therefore, I ask you all to have commit rights, so that we have two
committers here at IBM. I will mostly use it for the Portlet API (I think a
new branch is the best) and Ingo will do all other stuff from IBM, so that
the commits are consolidated.

Look in my other mail for an explanation of the sync and PortletConfig.

Stephan



Mime
View raw message