On 31 October 2011 10:59, Philippe Mouawad <philippe.moua...@gmail.com> wrote: > Hello Sebb, All, > I commited the fix to this feature implementing your proposal. > My tests show it fixes the 3 issues: > > - 51865 > - 51866 > - 51868 > > The think I dislike about it is the cast to LoopController in ThreadGroup > (but I think it is OK).
Probably. > As code is a bit complex it would be great if you could also test on you > side with all your knowledge of JMeter usages and internals. Looks OK, but I have not examined it in detail. > I don't think regressions can be introduced because it is only when Start > Next loop is used that changes do their job. > So regression would be on an already broken feature. Agreed. > I didn't mark issues as fixed yet nor did I change the comment in > changes.xml: > "Start next Loop option in Thread Group is broken, see Bugs (51868, 51866, > 51865)" Might be worth changing the bugs to "NEEDINFO" and asking the originators if the nightly builds have fixed the problems. > -- > Regards. > Philippe. > > > > > > On Tue, Oct 25, 2011 at 6:24 PM, sebb <seb...@gmail.com> wrote: > >> On 25 October 2011 17:14, Philippe Mouawad >> <p.moua...@ubik-ingenierie.com> wrote: >> > Hello Sebb, >> > I am looking at how to fix issues with "Start Next Loop". >> > Is there a way to get parent controller from child ? I don't see how ? >> >> Sorry, don't know offhand. >> >> > Would it be through a call to testTree.traverse and a SearchClass >> Controller >> > ? but we would also take into account controller that are down the >> > hierarchy. >> >> No idea - this part of JMeter is very complicated and not particularly >> well documented; I never fully got my head around it. >> >> The first stage might be to improve the Javadocs ... >> >> > Regards >> > Philippe >> > >> > >> > >> > On Wed, Oct 5, 2011 at 6:06 PM, <bugzi...@apache.org> wrote: >> > >> >> https://issues.apache.org/bugzilla/show_bug.cgi?id=51866 >> >> >> >> --- Comment #8 from Sebb <s...@apache.org> 2011-10-05 16:06:50 UTC --- >> >> (In reply to comment #7) >> >> > If I put JMeterContextService.getContext().isWithinRestartNextLoop() >> test >> >> > in fireIterEvents() instead, >> >> > do you see a case where it could fail ? >> >> >> >> Yes, if fireIterationStart() is called directly. >> >> But adding it to both won't necessarily help either, as that only fixes >> the >> >> issue with iteration listeners. >> >> >> >> But as I pointed out in Comment 3, it's not just the counter that >> >> misbehaves; >> >> the counter problem is just one symptom. >> >> >> >> I think the whole "Start next loop" code needs rewriting. >> >> >> >> Effectively the option means "go to end of loop" for each controller up >> to >> >> the >> >> Thread Group. [At least I assume this is the intention, as the option >> only >> >> appears on the Thread Group controller.] >> >> >> >> So we need to code the feature as if this has happened, and then >> everything >> >> else should happen naturally. >> >> >> >> -- >> >> Configure bugmail: >> >> https://issues.apache.org/bugzilla/userprefs.cgi?tab=email >> >> ------- You are receiving this mail because: ------- >> >> You voted for the bug. >> >> You are on the CC list for the bug. >> >> >> > >> > >> > >> > -- >> > Cordialement. >> > Philippe Mouawad. >> > Ubik-Ingénierie >> > >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@jakarta.apache.org >> For additional commands, e-mail: dev-h...@jakarta.apache.org >> >> > > > -- > Cordialement. > Philippe Mouawad. > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@jakarta.apache.org For additional commands, e-mail: dev-h...@jakarta.apache.org