Re: Add pg_file_sync() to adminpack

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: Add pg_file_sync() to adminpack
Дата
Msg-id 20200124055645.GC1581@paquier.xyz
обсуждение исходный текст
Ответ на Re: Add pg_file_sync() to adminpack  (Arthur Zakirov <zaartur@gmail.com>)
Ответы Re: Add pg_file_sync() to adminpack
Re: Add pg_file_sync() to adminpack
Список pgsql-hackers
On Fri, Jan 24, 2020 at 01:28:29PM +0900, Arthur Zakirov wrote:
> It is compiled and passes the tests. There is the documentation and it is
> built too without an error.
>
> It seems that consensus about the returned type was reached and I marked the
> patch as "Ready for Commiter".

+       fsync_fname_ext(filename, S_ISDIR(fst.st_mode), false, ERROR);
One comment here: should we warn better users in the docs that a fsync
failule will not trigger a PANIC here?  Here, fsync failure on heap
file => ERROR => potential data corruption.
--
Michael

Вложения

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

Предыдущее
От: Justin Pryzby
Дата:
Сообщение: Re: BUG #16109: Postgres planning time is high across version(Expose buffer usage during planning in EXPLAIN)
Следующее
От: Masahiko Sawada
Дата:
Сообщение: Re: Transactions involving multiple postgres foreign servers, take 2