On 6/7/07, Brent Worden <[EMAIL PROTECTED]> wrote:
I removed the use of the DistributionFactory and its dependency on
commons-discovery.
The approach I took was to add settable distribution fields where
needed as a replacement for invoking the factory. These fields always
default to our distribution implementations and can by altered via
constructors as well as setters.
Everything is checked in and critiques are welcome.
Thanks, Brent!
Looks great, modulo the two diff comments (NPE? and discovery
functionality). I am fine with the approach and +1 to applying the
same approach to the other abstract factory implementations. My only
substantive question is whether we should leave the discovery
mechanism working in the deprecated abstract factories.
Phil
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
