Re: [HACKERS] Cost of src/test/recovery and .../subscription tests
От | Andrew Dunstan |
---|---|
Тема | Re: [HACKERS] Cost of src/test/recovery and .../subscription tests |
Дата | |
Msg-id | fbaf0d9b-f8f8-8f05-94d5-44cf3a37503e@2ndQuadrant.com обсуждение исходный текст |
Ответ на | [HACKERS] Cost of src/test/recovery and .../subscription tests (Tom Lane <tgl@sss.pgh.pa.us>) |
Список | pgsql-hackers |
On 04/19/2017 01:28 PM, Tom Lane wrote: > So I updated longfin to the new release of the buildfarm client, > and was quite dismayed by the fact that its cycle time went > from 16 minutes to 24. Some of that might be random effects like > the state of the kernel disk caches, but a large chunk of it > --- over 5 minutes --- evidently is from src/test/recovery/, > which the buildfarm script didn't run before and now does. > > I am going to say flat out that that's unacceptably long for > a test script that will be run dozens of times a day by the > buildfarm. There isn't any other test script that takes more > than circa 90 seconds on that machine, and I don't think this > one should either. > > I'm a bit dubious about whether src/test/subscription is being > reasonably thrifty of test time either. Do we really need twice > the runtime of the entire core regression suite to test that? > > There was already a good argument not to enable the TAP tests > on slower buildfarm animals, and if nothing is done about these > additions, I'm afraid that such testing will be put permanently > out of reach. I don't even want to think about how long it > might take a CLOBBER_CACHE_ALWAYS animal to run these tests. You can disable the extra tests by adding this to the buildfarm command line: --skip-steps=misc-check cheers andrew > > regards, tom lane > > -- Andrew Dunstan https://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
В списке pgsql-hackers по дате отправления: