[jboss-dev-forums] [Design of Clustering on JBoss] - Re: API modifications for better extensibility ?

[bstansberry@jboss.com]

Hi Jean,

The code freeze for AS 5.1.0.GA is tomorrow and I'm on vacation, so the following is significantly colored by that. I can think about this more in a couple weeks when I get back, but once 5.1.0.GA is done there may not be a 5.2 as we move on to AS 6.

For AS 6 please open a JIRA to have another look at this design. I'm not comfortable with all these protected methods.

"deruelle_jean" wrote : 
  | *  initSnapshotManager to be made protected so that it can be overriden (so that we can set our own converged snapshotManager)

Done.

anonymous wrote : 
  | * getSnapshotManager() and setSnapshotManager() public methods or snapShotManager_ field to be made protected

Added a protected getter/setter. I don't want it public.

anonymous wrote : 
  | * installValves and installValve methods to be made protected so that they can be overriden (we need that to install our custom JVMRouteValve)

Done.

anonymous wrote : 
  | * Ideally, methods such as start/stopEmbedded and UnEmbedded and the methods they use internally should be made protected too

I renamed initCacheProxy() to initDistributedCacheManager() and made it protected. Also made initClusteredSessionNotificationPolicy() and initializeUnloadedSessions() protected.

Also added protected void startExtensions() and stopExtensions() which are a hook into the start/stop process for subclasses.

I do not want to expose startUnembedded. That is to support running in standalone Tomcat, which was a JIRA started in 2005 and never officially finished, not significantly tested in years etc. I don't want to make it part of this API. Does Mobicents run in standalone Tomcat?

Would prefer not exposing startEmbedded either.

If you *really need* start(Un)Embedded exposed, go ahead on Monday, but assume that there would be some change in AS 6; i.e. you and I will redesign this. And please be extremely careful as it's the code cutoff. :)

anonymous wrote : 
  | AbstractJBossCacheService
  | 
  | * make the storeSessionOwners protected so that we can reuse it
  | 

Feel free to make this a protected static method. However this change will not be in AS 5.1.0.GA as this would require a component update and we are past the deadline for component updates. A later Mobicents release of course could replace this jar with a later version. Which is why it's a separate jar. :)

anonymous wrote : 
  | IntervalSnapshotManager
  | 
  | * make the fields protected

This is a simple enough class, and you now have the hooks to install whatever SnapshotManager you like, so please just keep your own version.


The above changes are svn r88986 in Branch_5_x. Not ported to trunk yet.

View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4231472#4231472

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4231472