[jira] Updated: (POOL-175) Add Builders for Configuration classes

["Steve Siebert (JIRA)" <ji...@apache.org>]

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

Steve Siebert updated POOL-175:
-------------------------------

    Attachment: pool2_immutable_config_with_builders_v1

Attached is a half-patch toward this issue and also toward immutable issues/suggestions discussed in the mailing list (ref: https://issues.apache.org/jira/browse/POOL-173).  I don't intend this patch to be applied, as it is not complete, but would like to solicit feedback to this approach.  It makes the config instances immutable and adds the Builder patterns.

Implementation in the pool/factories is still debated (I'm for keeping a ref while Phil desires defensive copies ;) ).  Once I receive feedback on this partial, I will implement both with java 1.5 java.util.concurrent to see which approach would be preferred by the group =).

Continue on this approach =)

[+1] OK
[+-0] whatever
[-1] you suck: why....

Thanks =)

Steve



> Add Builders for Configuration classes
> --------------------------------------
>
>                 Key: POOL-175
>                 URL: https://issues.apache.org/jira/browse/POOL-175
>             Project: Commons Pool
>          Issue Type: Improvement
>    Affects Versions: 2.0
>            Reporter: Simone Tripodi
>            Assignee: Simone Tripodi
>             Fix For: 2.0
>
>         Attachments: pool2_immutable_config_with_builders_v1
>
>
> As suggested by Steven in the mailing-list thread:
> "Last suggestion/question is about making inner (public static final) Builder pattern classes within the concrete Config classes (and possibly defining an abstract <T extends Abstract*Config> create() method in the Abstract class).
> This would further simplify the programmatic creation of the Config classes."

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.