facebook/swift — pull request #60 by andrewcox (2013-02-08 17:28:13+00:00)
Title: Remove ThriftClientManagerProvider

Creating the ThriftClientManager through a provider makes it harder for Guice life cycle managemers to close() the manager in a @PreDestroy method. The only reason it was there was to enable passing a non-default maxFrameSize.

This change makes maxFrameSize a per-client configuration parameter, and binds ThriftClientManager directly (not through a provider).

Depends on https://github.com/facebook/nifty/pull/40