Re: [HACKERS][REVIEW] macaddr 64 bit (EUI-64) datatype support

Поиск
Список
Период
Сортировка
От Haribabu Kommi
Тема Re: [HACKERS][REVIEW] macaddr 64 bit (EUI-64) datatype support
Дата
Msg-id CAJrrPGd5gsCCezhCF783vCw25xjz=+Z5ft_dqALZLkKj+Fgf1A@mail.gmail.com
обсуждение исходный текст
Ответ на [HACKERS][REVIEW] macaddr 64 bit (EUI-64) datatype support  (Vitaly Burovoy <vitaly.burovoy@gmail.com>)
Ответы Re: [HACKERS][REVIEW] macaddr 64 bit (EUI-64) datatype support  (Stephen Frost <sfrost@snowman.net>)
Список pgsql-hackers


On Wed, Feb 1, 2017 at 6:27 AM, Vitaly Burovoy <vitaly.burovoy@gmail.com> wrote:
Hello,

I've reviewed the patch[1].

Result of testing:

make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           tested, passed
Documentation:            tested, passed


The patch introduce a new type macaddr8 for EUI-64 addresses[2]
(assuming OUI field is 24 bits wide) with EUI-48 (existing "macaddr"
type) interoperability.
It is a mostly copy-pasted macaddr implementation with necessary
changes for increased range.
Consensus was reached on such implementation in the current thread before.

There are two patch files for convenient reviewing: base macaddr8
implementation and its supporting in btree-gin and btree-gist indexes.

The patch:
* cleanly applies to the current master
(6af8b89adba16f97bee0d3b01256861e10d0e4f1);
* passes tests;
* looks fine, follows the PostgreSQL style guide;
* has documentation changes;
* has tests.

All notes and requirements were took into account and the patch was
changed according to them.
I have no suggestions on improving it.

The new status of this patch is: Ready for Committer

Thanks for the review.

Regards,
Hari Babu
Fujitsu Australia

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

Предыдущее
От: Haribabu Kommi
Дата:
Сообщение: Re: [HACKERS] [WIP]Vertical Clustered Index (columnar store extension)
Следующее
От: Kyle Gearhart
Дата:
Сообщение: Re: [HACKERS] libpq Alternate Row Processor