Summary: | wrong "size in bytes" when following redirections | ||
---|---|---|---|
Product: | JMeter - Now in Github | Reporter: | Dimitris Balaouras <dimitris> |
Component: | HTTP | Assignee: | JMeter issues mailing list <issues> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 2.3.4 | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All | ||
Attachments: | patch for correcting the parent's "size in bytes" when following redirections |
Description
Dimitris Balaouras
2010-07-06 12:39:31 UTC
There are two different usages for the constructor SampleResult(SampleResult res) - following redirects - downloading page resources I agree it seems wrong the way the sub-results are added currently, but I'm not yet sure of the best solution. Possibly the constructor should only create a clone of itself, and not add itself as a sub-result. The calling code would need to be changed accordingly, but would be easier to understand. Other samplers that call addSubResult() do so directly; it only seems to be the HTTP samplers that use the copy constructor. Is there a public URL that could be used to test against? Just noticed - google's redirect response includes text, so that can be used for testing. I've reworked the copy constructor to do a real copy and fixed the bytes problem (I hope!) In the process, the latency was also fixed for redirects. URL: http://svn.apache.org/viewvc?rev=961539&view=rev Log: Bug 49560 - wrong "size in bytes" when following redirections Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleResult.java jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java jakarta/jmeter/trunk/xdocs/changes.xml jakarta/jmeter/trunk/xdocs/usermanual/component_reference.xml The code will be in nightlies after r961539 - if possible, please could you try one and report back? Thanks! Hi Sebb, I've just svn up-ed my working copy to the HEAD revision and it looks nice! Many thanks for this quick fix! I have once more question though; I am wondering why did you set the latency of the parent to the latency of the first child? Shouldn't you count the overhead of the redirections in that figure? I would think that the latency of the parent element is from START until the first byte of the FINAL (landing) page. Thanks again! In the case of "download embedded", latency should definitely only be time to first response of parent page, but then the parent page is the first to be downloaded anyway. In the case of redirects, it's not so clear-cut, but I still think it makes sense to stick with the intial redirect latency, as that is how quickly the server has responded to the original request. hmm..indeed it's not so clear in case of redirects.. Maybe an "overall_latency" figure would make sense here. Anyways, thanks again for the fix! This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/2383 |