https://issues.apache.org/bugzilla/show_bug.cgi?id=41418

Sebb <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #10 from Sebb <[email protected]> 2009-11-17 17:28:02 UTC ---
I've been looking at the patch dated 2009-10-06 11:17 UTC.
[Not all that easy to follow, as there are lots and lots of spurious
white-space changes in the SampleResult part. Please try to submit minimum
necessary changes in future.]

There is one part of the patch I don't understand:

TransactionController.java
...
@@ -152,6 +164,8 @@
             // Assume success
             res.setSuccessful(true);
             res.sampleStart();
+            prevEndTime = res.getStartTime();
+            pauseTime = 0;
         }

It does not seem correct to use the StartTime as the initial value for the
previous End Time. Is the patch correct - if so why? If not, what should it be?

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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to