Ziborov Egor created AXIS2-5824:
-----------------------------------

             Summary: [Transport][JMS] Change thrown exception
                 Key: AXIS2-5824
                 URL: https://issues.apache.org/jira/browse/AXIS2-5824
             Project: Axis2
          Issue Type: Improvement
          Components: JMS transport
    Affects Versions: 1.7.4
            Reporter: Ziborov Egor
            Priority: Trivial


When we tries to lookup some destination for example "my.loc.dest" and it 
doesn't exist than we handle exception with message:
While trying to lookup 'dynamicQueues.local.rmk.access.SMKtoRMK.dq' didn't find 
subcontext 'dynamicQueues'. Resolved '' (for Weblogic)

I'd offer to change JMSUtils.lookupDestination from:
{code:java}
    public static Destination lookupDestination(Context context, String 
destinationName,
                                                String destinationType) throws 
NamingException {

        if (destinationName == null) {
            return null;
        }

        try {
            return JMSUtils.lookup(context, Destination.class, destinationName);
        } catch (NameNotFoundException e) {
            try {
                return JMSUtils.lookup(context, Destination.class,
                    
(JMSConstants.DESTINATION_TYPE_TOPIC.equalsIgnoreCase(destinationType) ?
                        "dynamicTopics/" : "dynamicQueues/") + destinationName);
            } catch (NamingException x) {
                log.warn("Cannot locate destination : " + destinationName);
                throw x;
            }
        } catch (NamingException e) {
            log.warn("Cannot locate destination : " + destinationName, e);
            throw e;
        }
    }
{code}

to:

{code:java}
    public static Destination lookupDestination(Context context, String 
destinationName,
                                                String destinationType) throws 
NamingException {

        if (destinationName == null) {
            return null;
        }

        try {
            return JMSUtils.lookup(context, Destination.class, destinationName);
        } catch (NameNotFoundException e) {
            try {
                return JMSUtils.lookup(context, Destination.class,
                    
(JMSConstants.DESTINATION_TYPE_TOPIC.equalsIgnoreCase(destinationType) ?
                        "dynamicTopics/" : "dynamicQueues/") + destinationName);
            } catch (NamingException x) {
                log.warn("Cannot locate destination : " + destinationName);
                throw e;
            }
        } catch (NamingException e) {
            log.warn("Cannot locate destination : " + destinationName, e);
            throw e;
        }
    }
{code}




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscr...@axis.apache.org
For additional commands, e-mail: java-dev-h...@axis.apache.org

Reply via email to