[
https://issues.apache.org/jira/browse/POOL-259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Phil Steitz updated POOL-259:
-----------------------------
Description:
The local variable waitTime used by borrowObject in both GOP and GKOP is not
updated unless borrowMaxWaitMillis is non-negative and blockWhenExhausted is
true. This effectively zeros the JMX client wait time properties for these
configurations. Even when there is a maxWait configured and the pool is set to
block when exhausted, what ends up being reported is only the wait time on the
instance queue. Note that this bug is JMX reporting only.
I am inclined to fix this by setting waitTime at the beginning and updating it
at the end (including activate and validate times). We should also add test
cases for this and the other stats exposed via JMX. Patches welcome, as it may
take me a little while to get to this.
was:The local variable waitTime used by borrowObject in both GOP and GKOP is
not updated unless borrowMaxWaitMillis is non-negative and blockeWhenExhausted
is true. This effectively zeros the JMX client wait time properties for these
configurations. Even when there is a maxWait configured and the pool is set to
block when exhausted, what ends up being reported is only the wait time on the
instance queue. I am inclined to fix this by setting waitTime at the beginning
and updating it at the end (including activate and validate times). We should
also add test cases for this and the other stats exposed via JMX. Patches
welcome, as it may take me a little while to get to this.
> Client wait time is not updated when max wait is configured to be unlimited
> ---------------------------------------------------------------------------
>
> Key: POOL-259
> URL: https://issues.apache.org/jira/browse/POOL-259
> Project: Commons Pool
> Issue Type: Bug
> Affects Versions: 2.2
> Reporter: Phil Steitz
>
> The local variable waitTime used by borrowObject in both GOP and GKOP is not
> updated unless borrowMaxWaitMillis is non-negative and blockWhenExhausted is
> true. This effectively zeros the JMX client wait time properties for these
> configurations. Even when there is a maxWait configured and the pool is set
> to block when exhausted, what ends up being reported is only the wait time on
> the instance queue. Note that this bug is JMX reporting only.
> I am inclined to fix this by setting waitTime at the beginning and updating
> it at the end (including activate and validate times). We should also add
> test cases for this and the other stats exposed via JMX. Patches welcome, as
> it may take me a little while to get to this.
--
This message was sent by Atlassian JIRA
(v6.2#6252)