I don't agree with changing module/config names just to denote we're
using a newer version of a depend. It had immediate impacts on
Daytrader, Samples and any Users' app that had to refer to server
plugins as a depend, like -
In daytrader/trunk -
daytrader-jetty/pom.xml - <artifactId>geronimo-activemq-ra</artifactId>
daytrader-tomcat/pom.xml - <artifactId>geronimo-activemq-ra</artifactId>
In samples/trunk -
samples/jms-mdb/jms-mdb-ear/pom.xml -
<artifactId>geronimo-activemq-ra</artifactId>
We could have used an artifactAlias, but that seems like a band-aide for
something that was easy to avoid and would have required entries for all
past release and potential 2.1.x future releases in order to not require
users to update their existing JMS apps before moving to 2.2.
Also, AMQ5 support was agreed upon as required for Geronimo 2.2 months
ago, so removing the AMQ4 bits just helps force the issue.
-Donald
David Jencks wrote:
On Dec 9, 2008, at 6:26 AM, Donald Woods wrote:
One of 2 good questions...
The ${activemqString} was put in there by Jencks, so I just kept using
it, but it should probably be removed.
Also, I don't like the renaming to activemq5-* for our modules/cars,
as now both our Samples and user apps will have to be updated to use
the new CAR names....
If there are no objections, I'll delete the old activemq plugins and
rename the activemq5 plugins, so we don't break everyone.
I strongly object. I'm still in the middle of working on this stuff so
even if I don't have any local uncommitted changes your unilateral
renames are going to cost me several hours rearranging my development
environment. I also think that there are good reasons to name the
integration code including the version of the integrated code. I'm not
convinced we will actually be able to get amq 5 into g 2.2 by
mid-january. I think there's value in letting people know which version
of the integrated product they are using. We did this for jetty6 and I
think it works OK. I think the latest actions on this were with jetty6
so the precedent is to use different names for different versions.
Changing this policy IMO ought to be discussed. Finally I think we may
want to support both amq 4 and 5 for 2.2.
What immediate problems was this causing you that you couldn't wait for
discussion? Since when is one hour sufficient time for discussion on a
change of this magnitude?
no thanks
david jencks
-Donald
Jarek Gawor wrote:
If we are switching to activemq5 why do we need this ${activemqString}
substitution?
Jarek
On Mon, Dec 8, 2008 at 6:59 PM, <[EMAIL PROTECTED]> wrote:
Author: djencks
Date: Mon Dec 8 15:59:58 2008
New Revision: 724558
URL: http://svn.apache.org/viewvc?rev=724558&view=rev
Log:
GERONIMO-4337 upgrade to activemq 5.2. Reduced console functionality
Added:
geronimo/server/trunk/plugins/activemq5/geronimo-activemq5/src/main/java/org/apache/geronimo/activemq/management/ActiveMQTransportConnector.java
(with props)
Modified:
geronimo/server/trunk/plugingroups/javaee5-jetty/pom.xml
geronimo/server/trunk/plugingroups/javaee5-jetty/src/main/history/dependencies.xml
geronimo/server/trunk/plugingroups/javaee5-tomcat/pom.xml
geronimo/server/trunk/plugingroups/javaee5-tomcat/src/main/history/dependencies.xml
geronimo/server/trunk/plugins/activemq5/activemq5-broker/src/main/history/dependencies.xml
geronimo/server/trunk/plugins/activemq5/activemq5-broker/src/main/plan/plan.xml
geronimo/server/trunk/plugins/activemq5/activemq5-portlets/src/main/java/org/apache/geronimo/console/jmsmanager/helper/AmqJMSMessageHelper.java
geronimo/server/trunk/plugins/activemq5/activemq5-portlets/src/main/java/org/apache/geronimo/console/jmsmanager/server/BaseJMSPortlet.java
geronimo/server/trunk/plugins/activemq5/activemq5-portlets/src/main/java/org/apache/geronimo/console/jmsmanager/server/JMSConnectorPortlet.java
geronimo/server/trunk/plugins/activemq5/activemq5-portlets/src/main/resources/jms-resource-providers.properties
geronimo/server/trunk/plugins/activemq5/activemq5-portlets/src/main/webapp/WEB-INF/view/jmsmanager/server/connector/normal.jsp
geronimo/server/trunk/plugins/activemq5/activemq5-portlets/src/main/webapp/WEB-INF/view/jmsmanager/server/normal.jsp
geronimo/server/trunk/plugins/activemq5/activemq5-server/pom.xml
geronimo/server/trunk/plugins/activemq5/geronimo-activemq5/src/main/java/org/apache/geronimo/activemq/BrokerServiceGBeanImpl.java
geronimo/server/trunk/plugins/activemq5/geronimo-activemq5/src/main/java/org/apache/geronimo/activemq/management/ActiveMQManagerGBean.java
geronimo/server/trunk/plugins/activemq5/pom.xml
geronimo/server/trunk/pom.xml
geronimo/server/trunk/testsuite/commands-testsuite/gshell/src/test/java/org/apache/geronimo/testsuite/gshell/deploy/DeployTest.java
geronimo/server/trunk/testsuite/console-testsuite/advanced/src/test/java/org/apache/geronimo/testsuite/console/JMSServerTest.java
geronimo/server/trunk/testsuite/enterprise-testsuite/jms-tests/jms-ear/src/main/resources/META-INF/geronimo-application.xml
geronimo/server/trunk/testsuite/web-testsuite/test-web-references/web-references-ear/src/main/resources/META-INF/geronimo-application.xml