Re: Online checksums patch - once again

Поиск
Список
Период
Сортировка
От Heikki Linnakangas
Тема Re: Online checksums patch - once again
Дата
Msg-id d2cf92c0-1c37-a4d4-e0cb-85110572a07e@iki.fi
обсуждение исходный текст
Ответ на Re: Online checksums patch - once again  (Heikki Linnakangas <hlinnaka@iki.fi>)
Ответы Re: Online checksums patch - once again  (Heikki Linnakangas <hlinnaka@iki.fi>)
Список pgsql-hackers
On 22/01/2021 13:55, Heikki Linnakangas wrote:
> I read through the latest patch,
> v31-0001-Support-checksum-enable-disable-in-a-running-clu.patch. Some
> comments below:

One more thing:

In SetRelationNumChecks(), you should use SearchSysCacheCopy1() to get a 
modifiable copy of the tuple. Otherwise you modify the tuple in the 
relcache as a side effect. Maybe that's harmless in this case, as the 
'relhaschecksums' value in the relcache isn't used for anything, but 
let's be tidy.

- Heikki



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

Предыдущее
От: Masahiro Ikeda
Дата:
Сообщение: RE: About to add WAL write/fsync statistics to pg_stat_wal view
Следующее
От: Daniel Gustafsson
Дата:
Сообщение: Re: mkid reference