You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hc.apache.org by "Sebb (JIRA)" <ji...@apache.org> on 2011/04/14 13:02:05 UTC
[jira] [Resolved] (HTTPCORE-254) Erratic results from metrics
[ https://issues.apache.org/jira/browse/HTTPCORE-254?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sebb resolved HTTPCORE-254.
---------------------------
Resolution: Fixed
Fix Version/s: 4.1.1
> Erratic results from metrics
> ----------------------------
>
> Key: HTTPCORE-254
> URL: https://issues.apache.org/jira/browse/HTTPCORE-254
> Project: HttpComponents HttpCore
> Issue Type: Bug
> Affects Versions: 4.1
> Reporter: Sebb
> Fix For: 4.1.1
>
>
> The values obtained from getMetrics are erratic. For example, the following code:
> DefaultHttpClient httpclient = new DefaultHttpClient();
> try {
> HttpRequestBase req = new HttpGet("http://wiki.apache.org/jakarta-jmeter/JMeterCommitters");
> for(int i = 0; i<3;i++) {
> HttpContext localContext = new BasicHttpContext();
> HttpResponse rsp = httpclient.execute(req, localContext);
> System.out.println(rsp.getStatusLine());
> HttpConnection conn = (HttpConnection) localContext.getAttribute(ExecutionContext.HTTP_CONNECTION);
> HttpConnectionMetrics metrics = conn.getMetrics();
> long hdr = metrics.getReceivedBytesCount();
> System.out.println("hdr "+hdr);
> HttpEntity entity = rsp.getEntity();
> // Thread.sleep(1500);
> if (entity != null) {
> EntityUtils.consume(entity);
> }
> long total = metrics.getReceivedBytesCount();
> System.out.println("tot "+total);
> metrics.reset();
> }
> } finally {
> // When HttpClient instance is no longer needed,
> // shut down the connection manager to ensure
> // immediate deallocation of all system resources
> httpclient.getConnectionManager().shutdown();
> }
> }
> May produce:
> HTTP/1.1 200 OK
> hdr 284
> tot 566
> HTTP/1.1 200 OK
> hdr 283
> tot 313
> HTTP/1.1 200 OK
> hdr 283
> tot 313
> Enabling the sleep produces more consistent (but still inaccurate) results:
> HTTP/1.1 200 OK
> hdr 284
> tot 10946
> HTTP/1.1 200 OK
> hdr 283
> tot 10945
> HTTP/1.1 200 OK
> hdr 283
> tot 10945
> Rather unexpected behaviour, but as it happens, a very simple cause:
> // AbstractSessionInputBuffer lines 184-189
> // If the remaining capacity is big enough, read directly from the
> // underlying input stream bypassing the buffer.
> if (len > this.minChunkLimit) {
> return this.instream.read(b, off, len);
> } else {
> // otherwise read to the buffer first
> The code also bypasses the metrics collection...
> Fixing this results in the following (with or without the sleep):
> {noformat}
> HTTP/1.1 200 OK
> hdr 284
> tot 16643
> HTTP/1.1 200 OK
> hdr 283
> tot 16642
> HTTP/1.1 200 OK
> hdr 283
> tot 16642
> {noformat}
> [Note that the slight variation in size is due to a minor change in the headers.]
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org