[PATCH] test-bundle2-format.t: force gc so a GeneratorExit will be thrown
Augie Fackler
raf at durin42.com
Wed Dec 30 15:50:01 CST 2015
> On Dec 30, 2015, at 1:07 PM, Sean Farley <sean at farley.io> wrote:
>
>
> Augie Fackler <raf at durin42.com> writes:
>
>> On Tue, Dec 22, 2015 at 10:47:09PM -0800, Bryan O'Sullivan wrote:
>>> # HG changeset patch
>>> # User Bryan O'Sullivan <bos at serpentine.com>
>>> # Date 1450853226 28800
>>> # Tue Dec 22 22:47:06 2015 -0800
>>> # Node ID b74ccb8db88fb4f333061215d67dba5065bc56ab
>>> # Parent 079eba011d07eb82319082116528abddb6a7b178
>>> test-bundle2-format.t: force gc so a GeneratorExit will be thrown
>>>
>>> PyPy has looser semantics than CPython for when a generator's close
>>> method will be called. Forcing the gc causes it to be called at
>>> the right moment.
>>
>> I'm a little weirded out by this fix, because it worries me that it is
>> going to choke us in weird ways in production. Can you allay that fear at all?
>
> This is just a test extension so it didn't set off any alarms for me, FWIW.
Right, but the test extension is emulating behavior I saw in real-world deployment, so I’m concerned that we might be masking a real problem on PyPy here.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20151230/83125450/attachment.pgp>
More information about the Mercurial-devel
mailing list