Message 209535 - Python tracker

Message209535

Author	pitrou
Recipients	kristjan.jonsson, larry, loewis, pitrou, serhiy.storchaka, vajrasky, vstinner
Date	2014-01-28.11:39:26
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1390909163.2293.1.camel@fsol>
In-reply-to	<1390908883.29.0.136463507827.issue20416@psf.upfronthosting.co.za>

Content
> Integers, without the patch: > > dumps v3: 62.8 ms > data size v3: 4882.8 kB > loads v3: 10.7 ms > > Integers, with the patch: > > dumps v3: 18.6 ms (-70%) > data size v3: 4882.8 kB (same size) > loads v3: 27.7 ms (+158%) As I wrote on python-dev, dumps performance isn't important for the pyc use case, but loads performance is. Therefore it appears this patch goes into the wrong direction. You are also ignoring the runtime benefit of sharing objects: smaller memory footprint of the actual Python process.

Content

> Integers, without the patch:
> 
>     dumps v3: 62.8 ms
>     data size v3: 4882.8 kB
>     loads v3: 10.7 ms
> 
> Integers, with the patch:
> 
>     dumps v3: 18.6 ms (-70%)
>     data size v3: 4882.8 kB (same size)
>     loads v3: 27.7 ms (+158%)

As I wrote on python-dev, dumps performance isn't important for the pyc
use case, but loads performance is. Therefore it appears this patch goes
into the wrong direction.

You are also ignoring the *runtime* benefit of sharing objects: smaller
memory footprint of the actual Python process.

History
Date	User	Action	Args
2014-01-28 11:39:26	pitrou	set	recipients: + pitrou, loewis, kristjan.jonsson, vstinner, larry, serhiy.storchaka, vajrasky
2014-01-28 11:39:26	pitrou	link	issue20416 messages
2014-01-28 11:39:26	pitrou	create