Hi, Maybe there's some issue with launchpad mirroring? Can you try the branch at http://bzr.squid-cache.org/bzr/squid3/trunk ?
On Thu, Feb 6, 2014 at 9:23 AM, Nikolai Gorchilov <n...@x3me.net> wrote: > I did bzr pool before diff. No changed files were found. Web interface also > shows no modified files - > http://bazaar.launchpad.net/~squid/squid/trunk/revision/13257 > > On Thu, Feb 6, 2014 at 8:24 AM, Kinkie <gkin...@gmail.com> wrote: >> It' the tip, a bzr pull should get it for you >> >> On Feb 6, 2014 2:45 AM, "Nikolai Gorchilov" <n...@x3me.net> wrote: >>> >>> Thanx, Alex, >>> >>> I'm ready to test it immediately, but unfortunately, due to unknown >>> reason r13257 doesn't show any file modifications. I.e. bzr diff >>> -r13256..13257 produces empty result. >>> >>> Best, >>> Niki >>> >>> On Wed, Feb 5, 2014 at 8:49 PM, Alex Rousskov >>> <rouss...@measurement-factory.com> wrote: >>> > >>> > On 02/03/2014 03:22 PM, Alex Rousskov wrote: >>> > > On 02/03/2014 01:55 PM, Amos Jeffries wrote: >>> > >> FWIW; this seems to be the same issue is under discussion in >>> > >> squid-users >>> > >> thread "rock store: a bug or ...?". Cc'ing Henrik Lidström and >>> > >> Nikolai >>> > >> Gorchilov. >>> > >> >>> > >> Henrik, Nokolai: if you could followup to squid-dev in future about >>> > >> this >>> > >> please. >>> > >>> > >>> > >> On 2014-02-04 05:25, Kinkie wrote: >>> > >>> Hi, >>> > >>> it appears that the large rock merge has introduced a regression, >>> > >>> probably related to the handling of keepalive and/or about finishing >>> > >>> to send an object to a client. >>> > >>> >>> > >>> I have confirmed that the issue is NOT present in revno 13200, but >>> > >>> it >>> > >>> is present in r13203 (r13201 and r13202 do not compile). >>> > >>> > > FWIW, if nobody volunteers earlier, I should be able to look at this >>> > > on >>> > > Wednesday. >>> > >>> > >>> > FYI: Kinkie and I fixed the bug we could reproduce. See trunk r13257. >>> > >>> > The bug was a side-effect of the following Collapsed Forwarding change: >>> > >>> > > revno: 12501.1.50 >>> > > committer: Alex Rousskov <rouss...@measurement-factory.com> >>> > > branch nick: collapsed-fwd >>> > > timestamp: Tue 2013-06-25 09:39:10 -0600 >>> > > message: >>> > > Mark client streams that sent everything as STREAM_COMPLETE. >>> > > >>> > > The old code used STREAM_UNPLANNED_COMPLETE if the completed stream >>> > > was >>> > > associated with a non-persistent connection, which did not make >>> > > sense to me >>> > > and, IIRC, led to store entry aborts even though the entries were >>> > > not damaged >>> > > in any way. >>> > > >>> > > This change may expose other subtle bugs, but none are known at this >>> > > time. >>> > > >>> > > See also: >>> > > http://www.squid-cache.org/mail-archive/squid-dev/200702/0017.html >>> > > http://www.squid-cache.org/mail-archive/squid-dev/201102/0210.html >>> > >>> > >>> > Cheers, >>> > >>> > Alex. >>> > -- Francesco
