Re: Minor clarifying changes to abbreviated key abort code comments

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: Minor clarifying changes to abbreviated key abort code comments
Дата
Msg-id CA+TgmobKwQEFuBYu4R5ShGM1au9Y56Rn4xb3h5=1ySiDJv0nEw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Minor clarifying changes to abbreviated key abort code comments  (Peter Geoghegan <pg@heroku.com>)
Список pgsql-hackers
On Tue, Nov 3, 2015 at 2:19 PM, Peter Geoghegan <pg@heroku.com> wrote:
> On Tue, Nov 3, 2015 at 11:15 AM, Robert Haas <robertmhaas@gmail.com> wrote:
>> OK, I see.  Fixing comments in the back-branches is not always a
>> productive use of time, and in general I might like it if you pushed
>> for such things less frequently.  But I've done it anyway in this
>> instance.
>
> I guess I favor doing it where the comment is actually wrong, which
> does apply here -- we don't *rely* on anything. We could very well
> abort when state is TSS_BUILDRUNS, but we elect not too, since
> TSS_BUILDRUNS is taken to mean that it's too late for aborting to be
> worth it.

Fair point.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



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

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: Minor clarifying changes to abbreviated key abort code comments
Следующее
От: Tom Lane
Дата:
Сообщение: Re: [PATCH] postgres_fdw extension support