Re: Fix unsigned output for signed values in SLRU error reporting

Поиск
Список
Период
Сортировка
От Pavel Borisov
Тема Re: Fix unsigned output for signed values in SLRU error reporting
Дата
Msg-id CALT9ZEGFM+fdKMJ-_-HP9mzm2nmrE7HjVritsz75EBiMbXGA+A@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Fix unsigned output for signed values in SLRU error reporting  (Andres Freund <andres@anarazel.de>)
Ответы Re: Fix unsigned output for signed values in SLRU error reporting  (Pavel Borisov <pashkin.elfe@gmail.com>)
Список pgsql-hackers
Afaics offset etc can't be negative, so I don't think this really improves
matters. I think there's quite a few other places where we use %u to print
integers that we know aren't negative.

If anything I think we should change the signed integers to unsigned ones. It
might be worth doing that as part of
https://www.postgresql.org/message-id/CAJ7c6TPDOYBYrnCAeyndkBktO0WG2xSdYduTF0nxq%2BvfkmTF5Q%40mail.gmail.com

That was one of my intentions in the mentioned patch, but I couldn't confirm that the page number (and offset) in SLRU was used signed not by purpose. Thank you for confirming this. I will try to replace int to unsigned where it is relevant in SLRU as part of the mentioned thread. Though it could be a big change worth a separate patch maybe.

Again thanks!
Pavel 

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

Предыдущее
От: Amit Kapila
Дата:
Сообщение: Re: Column Filtering in Logical Replication
Следующее
От: Dong Wook Lee
Дата:
Сообщение: Re: Add pg_freespacemap extension sql test