You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jakarta.apache.org by se...@apache.org on 2010/05/02 03:35:54 UTC

svn commit: r940140 - /jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java

Author: sebb
Date: Sun May  2 01:35:53 2010
New Revision: 940140

URL: http://svn.apache.org/viewvc?rev=940140&view=rev
Log:
Incomplete fix

Modified:
    jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java

Modified: jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java?rev=940140&r1=940139&r2=940140&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java (original)
+++ jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java Sun May  2 01:35:53 2010
@@ -144,16 +144,18 @@ public class PublisherSampler extends Ba
                     publisher.publish(tmsg);
                     buffer.append(tmsg);
                 }
-                result.sampleEnd();
                 result.setResponseCodeOK();
                 result.setResponseMessage(loop + " messages published");
                 result.setSuccessful(true);
                 result.setSamplerData(buffer.toString());
                 result.setSampleCount(loop);
+            } else {
+                result.setResponseMessage("Publisher not available - see log file");
             }
         } catch (Exception e) {
-            result.sampleEnd();
             result.setResponseMessage(e.toString());
+        } finally {
+            result.sampleEnd();            
         }
         return result;
     }