DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=41913>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=41913


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #19763|0                           |1
        is obsolete|                            |




------- Additional Comments From [EMAIL PROTECTED]  2007-03-23 01:54 -------
Created an attachment (id=19777)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=19777&action=view)
Updated patch

This is an updated patch, which adds an option to the transaction controller
gui.
The option is a radio button group :
"Samples of the transaction to include in the transaction sample"
with values "None", "Failing", and "All".
The default value is "None".

I also looked into adding an option "Abort transaction when sample fails", to
kind of simulate a rollback. It was not trivial to implement. If I was to
implement it, then I would have had to call the "Sampler.sample(null)" method
in the TransactionControll, i.e. duplicate the code in JMeterThread.run that
sets up the environment for the sampler, calls the sample method, and then
calls post processors, assertions and samplelisteners and so on. I don't think
that code should be duplicated into TransactionController.
If, however, the TransactionController somehow had been able to call the
"sample" method of it's sub samplers, then I think we could have avoided that
the subsamples reached the sample listeners. It would be kind of neat, because
then you could have an option to only see the transaction sample in the
visualizers.

Anyway, I hope the patch is ok now, and will investigate the issue of
"rollback" and visualizers only seeing transaction samples if you could give me
some advice on how to proceed. The patch adds useful functionality as it is
now.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to