[
https://issues.apache.org/jira/browse/POOL-175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Steve Siebert updated POOL-175:
-------------------------------
Comment: was deleted
(was: 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
>
>
> 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.
