Re: [PATCH] Align verify_heapam.c error message offset with test expectations

Поиск
Список
Период
Сортировка
От Neil Chen
Тема Re: [PATCH] Align verify_heapam.c error message offset with test expectations
Дата
Msg-id CAA3qoJkdMFcVN+G9uvYtY6LM=92vyiMK9Z351TBpOoKhQM2=CA@mail.gmail.com
обсуждение исходный текст
Ответ на [PATCH] Align verify_heapam.c error message offset with test expectations  ("zengman" <zengman@halodbtech.com>)
Ответы Re: [PATCH] Align verify_heapam.c error message offset with test expectations
Список pgsql-hackers
Дерево обсуждения
Hi zengman,

On Wed, Jan 21, 2026 at 10:37 PM zengman <zengman@halodbtech.com> wrote:

This indicates that for the test case at offnum == 36, the error message should report "offset 43" (the successor), not "offset 36" (the current tuple).
However, when I updated the test expectation from \d+ wildcard to the exact value offset 43, the test fails.
This makes me wonder whether the current code in verify_heapam.c (lines 777, 793, 799) should be using nextoffnum instead of ctx.offnum.


Fully agreed.
I think there's no need to modify the expectations. Seems the 0002 patch alone is sufficient.

В списке pgsql-hackers по дате отправления:

Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера