axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ben Reif (JIRA)" <>
Subject [jira] Created: (AXIS2-4188) JMSSender not extendable
Date Tue, 23 Dec 2008 20:10:44 GMT
JMSSender not extendable

                 Key: AXIS2-4188
             Project: Axis 2.0 (Axis2)
          Issue Type: Improvement
          Components: transports
    Affects Versions: 1.4, 1.4.1, 1.5
         Environment: All
            Reporter: Ben Reif

We need to extend the org.apache.axis2.transport.jms.JMSSender class so that we can add some
custom properties to the JMS Message (most likely via the MessageContext. However, the class
is full of private methods which must be copied into a sub-class, and it also uses other classes
such as JMSOutTransportInfo that only have package protected constructors. 

This class is clearly meant to be extended because you can redefine the implementation class
in the axis2.xml. But rather then just extending it and overridding a method, you have to
jump through hoops and copy a bunch of code to do so. 

In my opinion this is one of the most frustrating things about using open source code. Many
Apache (and Sun projects as well) have a bad habit of coding everything private, package protected,
or sometimes even making things final! Most open source projects these days are designed to
be extended, but coding things in this way defeats that purpose.  

Sorry for the long rant and rave, overall I think Axis2 is really great, but could you keep
this in mind moving forward :) , and at least maybe make this method protected in the next

JMSSender - 

protected Message createJMSMessage(MessageContext msgContext, Session session)  throws JMSException

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message