Re: Assertion Results bug? -- here's the fix 2 - Perhaps not!

2002-12-23 Thread Jonathan Carlson
At the moment, I'm rather baffled. As I started adding debug statements, it stopped happening (in non gui-mode; it never happens in gui mode as far as I can tell). I kept removing debug statements until there were none left and it still didn't happen. One thing I did discover before it stopped

Re: Assertion Results bug? -- here's the fix 2 - Perhaps not!

2002-12-23 Thread Jonathan Carlson
Thanks Scott. I didn't even realize there was a non-GUI mode, but I see it now in the documentation (since I am looking for it, I guess). I am able to duplicate it in non-gui mode, but not in gui mode. I will do some further looking into it. Thanks, Jonathan >>> [EMAIL PROTECTED] 12/22/02

Re: Assertion Results bug? -- here's the fix 2

2002-12-22 Thread Michal Kostrzewa
Hi Scott, > Can you please clarify the motivation for your query? Are you promoting a > "no patches without test cases" philosophy or is there something special > about this particular change that concerns you? I'm aware of reproducing thread related errors. I also trust Jonathan and you, my po

Re: Assertion Results bug? -- here's the fix 2 - Perhaps not!

2002-12-22 Thread Scott Eade
Jonathan, Your fix may function well under some circumstances, but is does not seem to cater for non-gui testing where I end up with a jmeter.log full of these (I don't think I checked the log when I was running in gui mode): 12/22/2002 9:08:27 PM ERROR - jmeter.engine: java.lang.ClassCastExcept

Re: Assertion Results bug? -- here's the fix 2

2002-12-20 Thread Jonathan Carlson
Thanks again, Scott, for preparing the patch. To answer your question on how pinned down the problem was, I was able to reproduce it on a run of a dozen or so requests that had 10 threads running. Which request and which thread that it occurred on was variable for each run, although it did ha

Re: Assertion Results bug? -- here's the fix 2

2002-12-20 Thread Scott Eade
Hi Michal, Short answer: No. Longer answer: a. in my experience issues relating to thread safety can sometimes be a little tricky to reproduce, let alone to reproduce consistently (I do note your use of 'almost unfailing'). I get the impression that Jonathan may in face have such an example, but

Re: Assertion Results bug? -- here's the fix 2

2002-12-19 Thread Michal Kostrzewa
Hi, Thanks very much for the patch. Do you have (Jonathan, Scott, ... ?) an 'almost unfailig' method for reproducing this error? (test scenario?) best regards Michal Kostrzewa -- To unsubscribe, e-mail: For additional commands, e-mail:

Re: Assertion Results bug? -- here's the fix 2

2002-12-19 Thread Scott Eade
I am using cvs to prepare the patch and am running my test script that is loaded with assertions to confirm correct behaviour. Cheers, Scott -- Scott Eade Backstage Technologies Pty. Ltd. http://www.backstagetech.com.au .Mac Chat/AIM: seade at mac dot com On 20/12/2002 9:56 AM, "Jonathan Carlso

Re: Assertion Results bug? -- here's the fix 2

2002-12-19 Thread Jonathan Carlson
Don't forget that my changes were based on 1.8 because that's all I could get for source (The nightly builds aren't getting put out there and I can't get to CVS). It may be safe to assume those methods didn't change in CVS after 1.8 but you should probably check that just to be sure if you haven

Re: Assertion Results bug? -- here's the fix 2

2002-12-19 Thread Scott Eade
Jonathan, 1. The compiler and matcher definitions were private. 2. I think you forgot to include: Public final static String TEST_PATTERNS = "Assertion.test_patterns"; I'll post the patch to bugzilla shortly. Cheers, Scott -- Scott Eade Backstage Technologies Pty. Ltd. http://www.backsta

Re: Assertion Results bug? -- here's the fix 2

2002-12-19 Thread Jonathan Carlson
Here are all the changes again, but with thread protection added for Perl5Compiler and pre-compilation of all regex patterns. I included the full methods since it seemed easier to do and because I added javadoc where there was none. Any news on getting the nightly builds going again? 1) The c

Re: Assertion Results bug? -- here's the fix

2002-12-19 Thread Jonathan Carlson
Hi Scott, Can you wait? I have some other issues that I'll send you. The Perl5Compiler isn't thread-safe either. I'll send everything all together. >>> [EMAIL PROTECTED] 12/19/02 12:03AM >>> Is anyone making this into a patch? I can do it and submit it to bugzilla if you like. Let me know.

Re: Assertion Results bug? -- here's the fix

2002-12-18 Thread Scott Eade
Is anyone making this into a patch? I can do it and submit it to bugzilla if you like. Let me know. Scott -- Scott Eade Backstage Technologies Pty. Ltd. http://www.backstagetech.com.au .Mac Chat/AIM: seade at mac dot com On 19/12/2002 5:15 AM, "Jonathan Carlson" <[EMAIL PROTECTED]> wrote: >

Re: Assertion Results bug? -- here's the fix

2002-12-18 Thread Jonathan Carlson
Success! The ORO Perl5Matcher is not thread-safe. I checked the mailing list and found some comments about Perl5Matcher needing to be accessed via a ThreadLocal instance in mult-threaded situations. This is what I did to ResponseAssertion to fix my problem: 1) I changed the "matcher" static f