[PATCH][experimental-ims] Fix conditional backend request in streaming mode
geoff at uplex.de
Tue Jan 10 20:57:22 CET 2012
-----BEGIN PGP SIGNED MESSAGE-----
On 01/10/12 04:41 PM, Jocelyn De La Rosa wrote:
> First of all thank you Geoff for updating the experimental-ims branch
You're welcome, but I hope the update didn't create a lot conflicts for
you. I didn't want to announce anything until after I'd done more stress
testing, but if I'm not the only person working on the branch I'll speak
So, the belated info is: the experimental-ims branch is now merged with
master as of yesterday, and passes all of the test cases. But I don't
think it's ready to pass all of the stress tests yet.
> After some tests, I found out that there was a problem using streaming
> (do_stream = true) with conditional backend request.
> Here's a small patch that fixes the bug :-)
Thanks, I had noticed the bug but hadn't been able to investigate it.
And didn't expect it to be so simple. %^) I'll try it out.
This has raised some questions in my mind about if and how IMS should
interact with streaming, something we haven't considered at all so far.
That may need some discussion, but first I'll try the patch.
** * * UPLEX - Nils Goroll Systemoptimierung
Tel +49 40 2880 5731
Mob +49 176 636 90917
Fax +49 40 42949753
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (SunOS)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
-----END PGP SIGNATURE-----
More information about the varnish-dev