This is an automated email from the ASF dual-hosted git repository.

fschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git


The following commit(s) were added to refs/heads/master by this push:
     new 5f1995d  Always call compiler.done(pack) on sampler when it is finished
5f1995d is described below

commit 5f1995de244986c820ed47028ceedf9167004673
Author: Felix Schumacher <[email protected]>
AuthorDate: Sat Oct 16 13:33:21 2021 +0200

    Always call compiler.done(pack) on sampler when it is finished
    
    When the sampler set sample to ignore, compiler.done(pack) was not called,
    which led to strange effects, as the thread loop thought, it was already
    initialized on the next iteration and did not set the new values before
    the next sample.
    
    Bugzilla Id: 65569
---
 src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java 
b/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java
index 9084fd4..26bca46 100644
--- a/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java
+++ b/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java
@@ -587,6 +587,7 @@ public class JMeterThread implements Runnable, 
Interruptible {
                 // This call is done by checkAssertions() , as we don't call it
                 // for isIgnore, we explictely call it here
                 setLastSampleOk(threadContext.getVariables(), 
result.isSuccessful());
+                compiler.done(pack);
             }
             // Check if thread or test should be stopped
             if (result.isStopThread() || (!result.isSuccessful() && 
onErrorStopThread)) {

Reply via email to