Uploaded image for project: 'Application Server 3  4  5 and 6'
  1. Application Server 3 4 5 and 6
  2. JBAS-3546

Change log level on CachedConnectionManager.closeConnection() to warn

    XMLWordPrintable

Details

    • Feature Request
    • Resolution: Won't Do
    • Trivial
    • None
    • JBossAS-4.0.4.GA
    • JCA service
    • None
    • Low
    • 0
    • 0% 0%

    Description

      The feature of the CachedConnectionManager automatically closing connections (and warning about the fact) is pretty useful; in fact, so useful that we would like to receive email notifications when it occurs. But, unfortunatelly, we are not receiving such notifications, as our log4j SMTP appender is set to the WARN threshold, while such warnings are being logged as info:

      if (e != null)
      log.info("Closing a connection for you. Please close them yourself: " + c, e);
      else
      log.info("Closing a connection for you. Please close them yourself: " + c);
      m.invoke(c, new Object[]{});
      }
      catch (Throwable t)

      { log.info("Throwable trying to close a connection for you, please close it yourself", t); }

      }
      catch (NoSuchMethodException nsme)

      { log.info("Could not find a close method on alleged connection objects. Please close your own connections."); }

      So, I'm proposing these loggings are changed from info() to warn() - I don't think such change will cause any 'logging havoc', as the messages are only logged when the CachedConnectionManager is set to debug them...

      Attachments

        Activity

          People

            weston.price Weston M. Price (Inactive)
            felipeal Felipe Leme (Inactive)
            Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 5 minutes
                5m
                Remaining:
                Remaining Estimate - 5 minutes
                5m
                Logged:
                Time Spent - Not Specified
                Not Specified