Re: Unit tests for SLRU
| От | Daniel Gustafsson |
|---|---|
| Тема | Re: Unit tests for SLRU |
| Дата | |
| Msg-id | 65EFC7A1-B9C5-42F5-9ADA-D88792AB9340@yesql.se обсуждение исходный текст |
| Ответ на | Re: Unit tests for SLRU (Aleksander Alekseev <aleksander@timescale.com>) |
| Ответы |
Re: Unit tests for SLRU
|
| Список | pgsql-hackers |
> On 15 Nov 2022, at 11:15, Aleksander Alekseev <aleksander@timescale.com> wrote: >> What do you think? > > It looks much better than before. I replaced strcpy() with strncpy() > and pgindent'ed the code. + /* write given data to the page */ + strncpy(TestSlruCtl->shared->page_buffer[slotno], data, BLCKSZ - 1); Would it make sense to instead use pg_pwrite to closer match the code being tested? > Other than that to me it looks ready to be committed. Agreed, reading over it nothing sticks out. -- Daniel Gustafsson https://vmware.com/
В списке pgsql-hackers по дате отправления: