[
https://issues.apache.org/jira/browse/AXIS2-5824?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15761638#comment-15761638
]
Hudson commented on AXIS2-5824:
-------------------------------
SUCCESS: Integrated in Jenkins build Axis2 #3612 (See
[https://builds.apache.org/job/Axis2/3612/])
AXIS2-5824: Avoid misleading JMS lookup exception referring to
dynamic(Queues|Topics). (veithen: rev 1775102)
* (edit)
axis2/modules/transport/jms/src/main/java/org/apache/axis2/transport/jms/JMSUtils.java
> [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: [email protected]
For additional commands, e-mail: [email protected]
