Commit f406269a authored by James Sedgwick's avatar James Sedgwick Committed by Sara Golemon

FileServer

Summary: Based on new FileRegion interface

Test Plan: ran it, streamed lots of files. I have found an obscure way to break it but I don't think it's a FileRegion issue, rather a pipeline/bootstrap issue. Still digging

Reviewed By: hans@fb.com

Subscribers: fugalh, folly-diffs@, jsedgwick, yfeldblum, chalfant

FB internal diff: D2088094

Signature: t1:2088094:1432758904:0f1e5765dbedea5ddbe047db50e7b632cfb0c053
parent 7dbfd2f8
...@@ -31,7 +31,7 @@ class PipelineManager { ...@@ -31,7 +31,7 @@ class PipelineManager {
virtual void deletePipeline(PipelineBase* pipeline) = 0; virtual void deletePipeline(PipelineBase* pipeline) = 0;
}; };
class PipelineBase { class PipelineBase : public DelayedDestruction {
public: public:
virtual ~PipelineBase() {} virtual ~PipelineBase() {}
...@@ -69,7 +69,7 @@ struct Nothing{}; ...@@ -69,7 +69,7 @@ struct Nothing{};
* If W is Nothing, write() and close() will be disabled. * If W is Nothing, write() and close() will be disabled.
*/ */
template <class R, class W = Nothing> template <class R, class W = Nothing>
class Pipeline : public PipelineBase, public DelayedDestruction { class Pipeline : public PipelineBase {
public: public:
Pipeline(); Pipeline();
~Pipeline(); ~Pipeline();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment