[squid-dev] [PATCH] Stuck transactions with large bodies

Amos Jeffries squid3 at treenet.co.nz
Sat Aug 1 02:50:05 UTC 2015


On 1/08/2015 12:40 p.m., Alex Rousskov wrote:
> Hello,
> 
>     The attached patch fixes trunk handling of large transactions broken
> by r14093 (Replace Packer object API with Packable API). I only had a
> few minutes to test this so further work in this direction may be necessary.
> 
> The patch also adds a missing virtual destructor to the new Packable API.
> 

Must be right. I also came up with exactly identical set of flush()
calls independently based on what you said in IRC. :-)

Sorry, I applied that already before seeing this mail.

Not sure about the destructor part. IIRC there were some issues when I
tried to add it during the Packable development. Though that may have
been with the SBuf bit of the project that is not in trunk.
Will test it out after the releases and apply if working.

Thanks.

Amos



More information about the squid-dev mailing list