User: pra     
  Date: 01/08/09 11:33:25

  Modified:    src/main/org/jboss/ejb/plugins/jms JMSContainerInvoker.java
  Log:
  Added better logging when new destinations are created instead of the exception 
stack wich was verry confusing; some better logging in the lifecycle methods (beans 
name)
  
  Revision  Changes    Path
  1.24      +11 -7     
jboss/src/main/org/jboss/ejb/plugins/jms/JMSContainerInvoker.java
  
  Index: JMSContainerInvoker.java
  ===================================================================
  RCS file: 
/cvsroot/jboss/jboss/src/main/org/jboss/ejb/plugins/jms/JMSContainerInvoker.java,v
  retrieving revision 1.23
  retrieving revision 1.24
  diff -u -r1.23 -r1.24
  --- JMSContainerInvoker.java  2001/08/06 22:55:00     1.23
  +++ JMSContainerInvoker.java  2001/08/09 18:33:25     1.24
  @@ -60,7 +60,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Sebastien Alborini</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Marc Fleury</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Jason Dillon</a>
  - * @version $Revision: 1.23 $
  + * @version $Revision: 1.24 $
    */
   public class JMSContainerInvoker
      implements ContainerInvoker, XmlLoadable
  @@ -102,6 +102,7 @@
      protected TransactionManager tm;
      protected ServerSessionPool pool;
      protected ExceptionListenerImpl exListener;
  +   protected String beanName;
   
      // Static --------------------------------------------------------
   
  @@ -252,8 +253,8 @@
         }
         catch (NamingException e) {
            // if the lookup failes, the try to create it
  -         log.warn("destination not found: " + jndiName, e);
  -
  +         log.warn("destination not found: " + jndiName + " reason: " + e);
  +         log.warn("creating a new temporary destination: " + jndiName);
            //
            // attempt to create the destination (note, this is very
            // very, very unportable).
  @@ -348,6 +349,9 @@
         
         // Queue or Topic
         String destinationType = config.getDestinationType();
  +
  +      // Bean Name
  +      beanName = config.getEjbName();
         
         // Is containermanages TX (not used?)
         boolean isContainerManagedTx = config.isContainerManagedTx();
  @@ -471,7 +475,7 @@
       */
      public void start() throws Exception
      {
  -      log.debug("Starting JMSContainerInvoker");
  +      log.debug("Starting JMSContainerInvoker for bean " + beanName);
         exListener = new ExceptionListenerImpl(this);
         connection.setExceptionListener(exListener);
         connection.start();
  @@ -482,7 +486,7 @@
       */
      public void stop()
      {
  -      log.debug("Stopping JMSContainerInvoker");
  +      log.debug("Stopping JMSContainerInvoker for bean " + beanName);
         // Silence the exception listener
         if (exListener != null) {
            exListener.stop();
  @@ -520,7 +524,7 @@
       */
      public void destroy()
      {
  -      log.debug("Destroying JMSContainerInvoker");
  +      log.debug("Destroying JMSContainerInvoker for bean " + beanName);
   
         // clear the server session pool (if it is clearable)
         try {
  @@ -632,7 +636,7 @@
               id = message.getJMSMessageID();
            } catch (JMSException e) {
               // what ?
  -            id = "JMSContainerInvoke";
  +            id = "JMSContainerInvoker";
            }
               
            // Invoke, shuld we catch any Exceptions??
  
  
  

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to