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

Phil Steitz resolved POOL-293.
------------------------------
    Resolution: Fixed

getEvictionPolicy exposed as protected in r1680279.


> Allow for simpler extending of EvictionConfig for use with custom 
> EvictionPolicies
> ----------------------------------------------------------------------------------
>
>                 Key: POOL-293
>                 URL: https://issues.apache.org/jira/browse/POOL-293
>             Project: Commons Pool
>          Issue Type: Improvement
>    Affects Versions: 2.3
>            Reporter: Nick Johnson
>            Priority: Minor
>
> Presently, EvictionConfig contains just three items of configuration: the 
> minEvictableIdleTimeMillis, softMinEvictableIdleTimeMillis, and minIdle.  In 
> the case of GenericObjectPool, these are set inside the evict() method and 
> passed along to the evictionPolicy.evict method.  The evictionPolicy cannot 
> be set directly, but rather is only instantiated by specifying a class name.  
> The evictionPolicy also cannot be read in a subclass.
> Consequently if it is desired to pass along some additional parameter to the 
> evictionPolicy, it has to either be a constant or a static variable applying 
> to every instance of the policy.  It isn't possible to easily pass it along 
> using a subclass of EvictionConfig because it's being created inside evict(), 
> and because there isn't a way to get a handle to the EvictionPolicy once it's 
> created, it also can't be set on the policy after the fact.
> It would be nice if BaseGenericObjectPool#getEvictionPolicy() were protected 
> instead of default security so that a subclass could invoke a setter on the 
> evictionPolicy instance to configure it, or if GenericObjectPool#evict could 
> call a protected method like createEvictionConfigInstance(...), so that a 
> subclass could provide a custom EvictionConfig instance containing the 
> additional configuration for a subclassed EvictionPolicy to use.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to