[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jboss-cvs-commits
Subject:    [jboss-cvs] jboss-jms/src/main/org/jboss/messaging/core/plugin  ...
From:       Ovidiu Feodorov <ovidiu.feodorov () jboss ! com>
Date:       2006-01-31 23:37:16
Message-ID: E1F453k-000225-Q0 () committer01 ! frg ! pub ! inap ! atl ! jboss ! com
[Download RAW message or body]

  User: ovidiu  
  Date: 06/01/31 18:37:16

  Modified:    src/main/org/jboss/messaging/core/plugin 
                        JDBCTransactionLog.java
  Log:
  - Clarified the distinction between destination undeployment/re-deployment and destination drop.
    Added a test case that checks whether the current convention is enforced.
    Currently, is not possible to drop destinations, they will be always "re-deployed" or "activated".
    See http://jira.jboss.org/jira/browse/JBMESSAGING-220
  - Move various tests under the correct packages.
  
  Revision  Changes    Path
  1.7       +10 -3     jboss-jms/src/main/org/jboss/messaging/core/plugin/JDBCTransactionLog.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: JDBCTransactionLog.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-jms/src/main/org/jboss/messaging/core/plugin/JDBCTransactionLog.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -b -r1.6 -r1.7
  --- JDBCTransactionLog.java	26 Jan 2006 06:20:54 -0000	1.6
  +++ JDBCTransactionLog.java	31 Jan 2006 23:37:16 -0000	1.7
  @@ -64,9 +64,9 @@
    * @author <a href="mailto:tim.fox@jboss.com">Tim Fox</a>
    * @author <a href="mailto:adrian@jboss.org">Adrian Brock</a>
    *
  - * @version <tt>$Revision: 1.6 $</tt>
  + * @version <tt>$Revision: 1.7 $</tt>
    *
  - * $Id: JDBCTransactionLog.java,v 1.6 2006/01/26 06:20:54 ovidiu Exp $
  + * $Id: JDBCTransactionLog.java,v 1.7 2006/01/31 23:37:16 ovidiu Exp $
    */
   public class JDBCTransactionLog extends ServiceMBeanSupport implements TransactionLog
   {
  @@ -544,6 +544,7 @@
         Connection conn = null;
         PreparedStatement ps = null;
         TransactionWrapper wrap = new TransactionWrapper();
  +      boolean success = false;
   
         try
         {
  @@ -554,6 +555,7 @@
            ps.setString(1, (String)channelID);
            
            ps.executeUpdate();
  +         success = true;
         }
         catch (Exception e)
         {
  @@ -562,6 +564,11 @@
         }
         finally
         {         
  +         if (log.isTraceEnabled())
  +         {
  +            String s = JDBCUtil.statementToString(deleteChannelMessageRefs, channelID);
  +            log.trace(s + (success ? " successful" : "failed"));
  +         }
            if (ps != null)
            {
               try
  
  
  


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
jboss-cvs-commits mailing list
jboss-cvs-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jboss-cvs-commits
[prev in list] [next in list] [prev in thread] [next in thread]