Re: Rename some rel truncation related constants at the top of vacuumlazy.c

Поиск
Список
Период
Сортировка
От Peter Geoghegan
Тема Re: Rename some rel truncation related constants at the top of vacuumlazy.c
Дата
Msg-id CAH2-Wz=2BDRCrU8+XGGrK1BaJ3Ywez7_Q7st_HG52et_+9p4iA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Rename some rel truncation related constants at the top of vacuumlazy.c  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Mon, Jul 18, 2022 at 8:55 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Um ... you seem to have removed some useful comments?

I don't think that the stuff about making them into a GUC is useful myself.

> Personally I wouldn't do this, as I don't think the renaming
> brings much benefit, and it will create a hazard for back-patching
> any fixes that might be needed in that code.  I'm not hugely upset
> about it, but that's the way I'd vote if asked.

In that case I withdraw the patch.

FWIW I wrote the patch during the course of work on new feature
development. A patch that added a couple of similar constants a bit
further down. Seemed neater this way, but it's certainly not worth
arguing over.

-- 
Peter Geoghegan



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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Rename some rel truncation related constants at the top of vacuumlazy.c
Следующее
От: John Naylor
Дата:
Сообщение: Re: [PoC] Improve dead tuple storage for lazy vacuum