Re: [HACKERS] Tuple sort is broken. It crashes on simple test.

Поиск
Список
Период
Сортировка
Искать
От
Peter Geoghegan
Тема
Re: [HACKERS] Tuple sort is broken. It crashes on simple test.
Дата
Msg-id
CAM3SWZQ9MkChJCFb3c_wLDhCH3fP=VPJ-XMU01nekkU+QaeRCA@mail.gmail.com
Ответ на
Список
Дерево обсуждения
[HACKERS] Tuple sort is broken. It crashes on simple test. Mithun Cy <mithun.cy@enterprisedb.com>
Re: [HACKERS] Tuple sort is broken. It crashes on simple test. Michael Paquier <michael.paquier@gmail.com>
Re: [HACKERS] Tuple sort is broken. It crashes on simple test. Peter Geoghegan <pg@heroku.com>
Re: [HACKERS] Tuple sort is broken. It crashes on simple test. Tom Lane <tgl@sss.pgh.pa.us>
Re: [HACKERS] Tuple sort is broken. It crashes on simple test. Pavel Stehule <pavel.stehule@gmail.com>
Re: [HACKERS] Tuple sort is broken. It crashes on simple test. Peter Geoghegan <pg@heroku.com>
Re: [HACKERS] Tuple sort is broken. It crashes on simple test. Pavel Stehule <pavel.stehule@gmail.com>
On Mon, Jan 16, 2017 at 10:38 AM, Pavel Stehule  wrote:
> Should not be enhanced regress tests too?

We already have coverage of multi-pass external tuplesorts, as of a
few months back. That didn't catch this bug only because it was a
pass-by-value datum tuplesort. The relevant RELEASE_SLAB_SLOT() call
does have line coverage already.

I wouldn't object to adding a test case that would have exercised this
bug, too. It took me a while to talk Tom into the test that was added
several months back, which discouraged me from adding another test
case here. (There were concerns about the overhead of an external sort
test on slower buildfarm animals.)

-- 
Peter Geoghegan


В списке pgsql-hackers по дате отправления
От: Alvaro Herrera
Дата:
От: Andres Freund
Дата:
FAQ