hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Seth Weiner (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HTTPCORE-311) Retrieval of primitive connection pool stat values
Date Wed, 19 Sep 2012 19:02:08 GMT

     [ https://issues.apache.org/jira/browse/HTTPCORE-311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Seth Weiner updated HTTPCORE-311:

    Attachment: httpcomponents-monitoring-patch2.diff

Okay, here's another diff. A few things to note:

I used AtomicIntegers for each of the 3 stat values rather than a single Lock. I'm not sure
if there's a strong argument either way for that, but my rationale was that they're often
modified independent of one another so why bother locking all of them.

I didn't change the implementation of getTotalStats() largely to demonstrate the behaviors
matched between the values in the PoolStats object and the individual metrics. However, that
can certainly be changed if you prefer.
> Retrieval of primitive connection pool stat values
> --------------------------------------------------
>                 Key: HTTPCORE-311
>                 URL: https://issues.apache.org/jira/browse/HTTPCORE-311
>             Project: HttpComponents HttpCore
>          Issue Type: New Feature
>          Components: HttpCore
>    Affects Versions: 4.2.1
>            Reporter: Seth Weiner
>         Attachments: httpcomponents-monitoring-patch2.diff, httpcomponents-monitoring-patch.diff,
PollingPrimitivePoolControl.java, PollingPrimitivePoolControlTest.java, PrimitiveConnPoolControl.java
> The org.apache.http.pool.ConnPoolControl interface exposes valuable runtime metrics for
a connection pool's individual routes and global values. However, it uses a bean class, org.apache.http.pool.PoolStats,
to exposes the global values. This is problematic when trying to monitor these connection
pool values at runtime since PoolStats is not Serializable and many jmx monitoring agents
only support primitive values well. The value of the PoolStats class seems to be in trying
to minimize the amount of locking of the pool required to get stats.
> The attached patch includes an interface that extends ConnPoolControl to add four methods
to return the primitive value for global pool stats. There's an implementation of this interface
that will delegate all of the ConnPoolControl methods to a delegate instance, and support
retrieving the primitive global values by periodically retrieving and caching a PoolStats
instance internally. This minimizes the amount of locking required while also making it easy
to monitor these values.
> It would probably be ideal to incorporate this functionality directly into the existing
ConnPoolControl interface, but the attached code provides a non-invasive way to achieve the
desired effect.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
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

View raw message