update thrift to use managed connection
authorDave Watson <davejwatson@fb.com>
Wed, 24 Sep 2014 17:52:50 +0000 (10:52 -0700)
committerAnton Likhtarov <alikhtarov@fb.com>
Fri, 26 Sep 2014 22:23:37 +0000 (15:23 -0700)
commitd8e719111859150a8d310069513dd5099a623f8f
treea5a9695e6331a82a910ee8012bee0c92022e9a8f
parentf760023d339249ca3f6e40ebee69128d9850f925
update thrift to use managed connection

Summary:
Use the new managed connection code in folly.  Cleans things up a fair bit

The only catch was the current ConnectionManager code uses raw pointers, thrift expects shared_ptr so the request object can keep the channel around until the requests are finished

Test Plan: fbconfig -r thrift/lib/cpp2; fbmake runtests

Reviewed By: afrind@fb.com

Subscribers: trunkagent, njormrod, doug, fugalh, alandau, bmatheny, dcsommer, afrind

FB internal diff: D1519923

Tasks: 5002343
folly/experimental/wangle/ConnectionManager.h