commit c4a90ecb0af90d175468a4bdf43cd84675543294
author scottb@google.com <scottb@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Wed Mar 25 18:12:37 2009 +0000
committer scottb@google.com <scottb@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Wed Mar 25 18:12:37 2009 +0000
tree ca1c5d822ffa70d4810002fe03980f569fd2049e
parent a41864d8e4635f9d2ff9a749dca1d16b618f9003

Removes the PermutationWorkerFactory.lazyFactories cache.

The problem with caching these as static state is that the Factories themselves do not appear to be stateless.  In particular, ExternalPermutationWorkerFactory has instance state related to accepting a particular number of connections.  This is neither threadsafe and also makes it impossible to reuse.  We can revisit this change if we decide that PWFs should be both threadsafe and reusable (or stateless).

Patch by: scottb, spoon
Review by: bobv (TBR), spoon

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@5080 8db76d5a-ed1c-0410-87a9-c151d255dfc7