Bug 64381 - PreciseThroughputTimer: On termination, log message contains negative value
Summary: PreciseThroughputTimer: On termination, log message contains negative value
Status: RESOLVED FIXED
Alias: None
Product: JMeter - Now in Github
Classification: Unclassified
Component: Main (show other bugs)
Version: 5.2.1
Hardware: All All
: P2 minor (vote)
Target Milestone: JMETER_5.3.0
Assignee: JMeter issues mailing list
URL:
Keywords: FixedInTrunk
Depends on:
Blocks:
 
Reported: 2020-04-26 16:07 UTC by Philippe Mouawad
Modified: 2020-04-26 16:09 UTC (History)
1 user (show)


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Mouawad 2020-04-26 16:07:30 UTC 
When Timer needs to terminate thread, it logs:

- 2020-04-22 22:33:45,487 INFO o.a.j.t.JMeterThread: Stopping Thread: org.apache.jorphan.util.JMeterStopThreadException: The thread is scheduled to stop in -70320 ms and the throughput timer generates a delay of 159461. JMeter (as of 4.0) does not support interrupting of sleeping threads, thus terminating the thread manually.


This value -70320 is incorrect.
Comment 1 Philippe Mouawad 2020-04-26 16:09:24 UTC 
This is an automated email from the ASF dual-hosted git repository.

pmouawad 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 0772c1d  Bug 64381 - PreciseThroughputTimer: On termination, log message contains negative value
0772c1d is described below

commit 0772c1d97e49aa6c5e615f1fc02ba7bd215c710b
Author: pmouawad <p.mouawad@ubik-ingenierie.com>
AuthorDate: Sun Apr 26 18:08:42 2020 +0200

    Bug 64381 - PreciseThroughputTimer: On termination, log message contains
    negative value

    Also avoid calling System.currentTimeMillis() too much times
---
 .../jmeter/timers/poissonarrivals/PreciseThroughputTimer.java | 11 ++++++-----
 xdocs/changes.xml                                             |  1 +
 2 files changed, 7 insertions(+), 5 deletions(-)
Comment 2 The ASF infrastructure team 2022-09-24 20:38:19 UTC 
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/5298