Re: Misplaced BKI entries in pg_amproc.h

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: Misplaced BKI entries in pg_amproc.h
Дата
Msg-id 20140131210651.GP19957@momjian.us
обсуждение исходный текст
Ответ на Misplaced BKI entries in pg_amproc.h  (Antonin Houska <antonin.houska@gmail.com>)
Список pgsql-hackers
On Thu, Aug  1, 2013 at 10:14:43AM +0200, Antonin Houska wrote:
> While checking something, I noticed that opfamilies 3626, 3683, 3901
> (all btree AM), 3903 (hash) and 3919 (gist) are all defined in the
> section marked as "gin".
>
> (I'm not sure if it helps to deliver a patch - it may be easier for
> the committer to move the items himself than to check if the diff is
> correct)

You are absolutely correct.  I checked your report by comparing
src/include/catalog/pg_amproc.h against this query:

    SELECT pg_opfamily.oid, pg_opfamily.*, pg_am.amname
    FROM pg_opfamily join pg_am ON (pg_opfamily.opfmethod = pg_am.oid)
    ORDER BY 2, 1;

and you are right that the ones you mentioned were in the wrong section.
I also reordered a few so the entries in each section were in increasing
order.

Attached patch applied.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + Everyone has their own god. +

Вложения

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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: [COMMITTERS] pgsql: libpq: Support TLS versions beyond TLSv1.
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Recovery inconsistencies, standby much larger than primary