On Sat, Feb 15, 2020 at 1:30 PM Felix Schumacher <
felix.schumac...@internetallee.de> wrote:

>
> Am 15.02.20 um 11:26 schrieb Philippe Mouawad:
> > Hello,
> > Working on Bug 64142, and looking into JMeterThread class, I notice
> > something that might not be correct.
> >
> > It seems we fill in total number of thread and group number of thread
> after
> > sampling has occurred:
> >
> >    -
> >
> https://github.com/apache/jmeter/blob/master/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java#L564
> >
> > Isn't it too late ? Shouldn't we be doing that before this line?
>
> Where do you want to put it? You mention L564 twice.
>
oups
I meant to store total and group thread  here:
https://github.com/apache/jmeter/blob/master/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java#L555

>
> Felix
>
> >    -
> >
> https://github.com/apache/jmeter/blob/master/src/core/src/main/java/org/apache/jmeter/threads/JMeterThread.java#L564
> >
> >
>


-- 
Cordialement.
Philippe Mouawad.

Reply via email to