Pipeline OutputBufferingHandler
authorDave Watson <davejwatson@fb.com>
Wed, 8 Apr 2015 20:54:59 +0000 (13:54 -0700)
committerViswanath Sivakumar <viswanath@fb.com>
Fri, 10 Apr 2015 03:35:11 +0000 (20:35 -0700)
commit8e2ec4c39710bf7012ddc69ae3cc55ed45ccd90c
tree681d12341280e944ddfc8b8355f1bcbbd9087e34
parent51026ab6092a4bd8d7dc295d4b19245d63babdc6
Pipeline OutputBufferingHandler

Summary: Part 2 of pipeline.  Use OutputBufferingHandler, which is exactly the same output buffering thrift does currently.

Test Plan:
A couple canaries, unittests.
Specific issues of previous pipeline diffs I think are fixed already and should be fine with this one:
* Previously there were missing DestructorGuards.  I don't think anything in OutputBUfferHandler needs a DG
* previously broke http GET handling.  There is a unittest for it now.

Canary

Reviewed By: alandau@fb.com

Subscribers: doug, fugalh, alandau, bmatheny, mshneer, folly-diffs@, jsedgwick, yfeldblum, chalfant, vloh, andreib

FB internal diff: D1970949

Signature: t1:1970949:1428360621:a0db142032894525f69c2b144cf946d63f790fe1
folly/wangle/channel/OutputBufferingHandler.h