Phil Steitz <phil.ste...@gmail.com> wrote:

>I see this now in GOP javadoc for numActive.
>
> * The combined count of the currently active objects and those in the
>     * process of being created. Under load, it may exceed {@link
>#_maxActive}
>     * but there will never be more than {@link #_maxActive} created
>at any one
>     * time.
>
>[performance] still checks to make sure this does not happen and I
>have not seen it yet in my load tests.  I don't personally like the
>idea of allowing the number of instances being created + under
>management to exceed maxActive.  This could cause resource problems
>for systems that experience load spikes and if it is allowed "a
>little" then on loaded systems that can quickly escalate to "a
>lot".  I don't think we should back off of this invariant contract. 
>Could be I am misunderstanding the comment and what it really means
>to say is that numActive can overstate the instance count in certain
>circumstances.  Is that what is meant?
>
>Also, I think it would be great to change the name of this member
>variable and the numActive property on the Deque in GKOP.  The name
>is misleading as it is really idle + active + being created.  Maybe
>something like instanceCount.
>
>Phil
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
>For additional commands, e-mail: dev-h...@commons.apache.org

The naming could be better and there is probably a case for two attributes.

The short version is:
- there will never be more than maxActive objects created / in creation
- under load numActive may exceed maxActive for a very short time but it will 
be reduced again a few lines of code later before the object creation process 
starts.

There is actually a bunch of attribute renaming that needs to be done.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to