Re: PATCH: pg_basebackup (missing exit on error)

Поиск
Список
Период
Сортировка
От Fujii Masao
Тема Re: PATCH: pg_basebackup (missing exit on error)
Дата
Msg-id CAHGQGwFrnPMhpJZT31J4JtCfSrMJfvgtESQVkjXxHhROd0A3Eg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: PATCH: pg_basebackup (missing exit on error)  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
Дерево обсуждения 
On Wed, Mar 28, 2012 at 9:40 PM, Robert Haas <robertmhaas@gmail.com> wrote:
> I think it would make sense to rearrange that so that we don't have
> two tests for ztarfile != NULL; do that test first, and then if it
> fails, do the strcmp after that.

Makes sense.

> Also, if we're going to test the return value of fclose(), shouldn't
> we also be checking the return value of gzclose()?

Yes, we should.

Attached patch does the above two changes.

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
Вложения

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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Re: pg_stat_statements normalisation without invasive changes to the parser (was: Next steps on pg_stat_statements normalisation)
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Re: [COMMITTERS] pgsql: pg_test_timing utility, to measure clock monotonicity and timing