Re: [PATCH] ltree, lquery, and ltxtquery binary protocol support

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [PATCH] ltree, lquery, and ltxtquery binary protocol support
Дата
Msg-id 13998.1585777035@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH] ltree, lquery, and ltxtquery binary protocol support  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [PATCH] ltree, lquery, and ltxtquery binary protocol support
Список pgsql-hackers
I wrote:
> Fortunately for the odds of getting this patch accepted, we just
> pushed an ALTER TYPE improvement that will solve your problem [1].
> Please replace ltree--1.2.sql with an upgrade script that uses
> that, and resubmit.

I decided it would be a shame for this to miss v13, seeing that
(a) it'd provide real-world use of that ALTER TYPE code, and
(b) we already bumped ltree's extension version for v13.

So I went ahead and rebased this, reviewed and pushed it.  The
main non-cosmetic thing I changed, other than using ALTER TYPE,
was that you had the output functions looking like this:

    pq_sendtext(&buf, res, pg_mbstrlen(res));

That's just wrong; it should be plain strlen().  Did you copy
that coding from someplace?  I could not find any similar
occurrences in our code tree.

            regards, tom lane



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

Предыдущее
От: David Steele
Дата:
Сообщение: Re: backup manifests
Следующее
От: Tomas Vondra
Дата:
Сообщение: Re: [PATCH] Incremental sort (was: PoC: Partial sort)