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.