Code review comment for lp:~adeuring/launchpad/bug-507642

Revision history for this message
Abel Deuring (adeuring) wrote :

Hi Jeroen,

I added an explicit test that the collections are labeled as "do not snapshot". But I did not run the "full test" of too many subscritpions with the real hard limit, as you suggested: Doing this takes ca 80 second on my machine,even when factory.makePersonNoCimmit() is used. BTW, according to top(1), most time is spent in Python, not in Postgres during this test.

« Back to merge proposal