river-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter <j...@zeus.net.au>
Subject Re: [Discuss] Drop support for Activation?
Date Fri, 13 Nov 2015 23:58:40 GMT
I'd like to propose Removing it from River and placing it into a separate module or subproject.

I'd like to see this done as part of the modular build effort.

I think this should be part of River 3.1



Sent from my Samsung device.
  Include original message
---- Original message ----
From: Greg Trasuk <trasukg@trasukcom>
Sent: 14/11/2015 01:21:47 am
To: dev@river.apache.org
Cc: user@river.apache.org
Subject: [Discuss] Drop support for Activation?

Hello all: 

Last week I asked about removing activation from River, both the 2.2 and 3.0 branches.  There didn’t seem to be a lot of anti-removal feeling, so I’d like to formally propose removing Activation.  There are a couple of other things that we could possibly remove, like JRMP support (i.e. pre-compiled proxy classes), but we should probably discuss those separately.

The main reason for this is that unused code still requires maintenance and increases the chance of bugs.  Also I think that as we go forward with refactoring, renaming, restructuring the build and so on, it seems wasteful to do that work on code that isn’t actually in use.

Obviously, the code remains in Subversion and in the 2.2.2 release, so if someone wants to get it back, we (or they) could package it into a different deliverable.  But I wouldn’t plan on doing that unless there’s actual demand for it.

My thought is to put this out there for discussion - If there is consensus after a few days I’ll call a lazy-consensus vote.   I’ll be happy to do the work in the 2.2 branch.

So, I propose to drop support for the following: 

Activation -  

Norm / LeaseRenewalService - is pretty much unneeded without activation 

Activatable implementation of the infrastructure services 
Starter for Activatable Services 
QA Harness classes that test any of the above. 


Greg Trasuk 

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message