Patch review for limiting backend retry on post

[Martin Blix Grydeland]

Thanks for the feedback Poul-Henning. I've implemented what you suggested in
the attached two patches.

First changes FetchReqBody to report what it's done, and then only allow a
retry if it reported no body.

Second sets the retry variable only if we are doing a GET or HEAD.

Another point for making Varnish buffer request bodies before doing any
backend work at all is dealing with resource hungry backends and slow
sending clients. We would then be able to buffer the data before tying up a
backend resource and send it all in one fast swoop.

-Martin

-- 
Martin Blix Grydeland
Varnish Software AS
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20110803/202f1c6e/attachment-0003.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Make-FetchReqBody-report-what-was-done-and-don-t-all.patch
Type: application/octet-stream
Size: 2324 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20110803/202f1c6e/attachment-0006.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Only-retry-a-recycled-backend-connection-on-GET-HEAD.patch
Type: application/octet-stream
Size: 820 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20110803/202f1c6e/attachment-0007.obj>