Re: pg_lzcompress patch for 8.3, 8.2 branch

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: pg_lzcompress patch for 8.3, 8.2 branch
Дата
Msg-id 5878.1212009110@sss.pgh.pa.us
обсуждение исходный текст
Ответ на pg_lzcompress patch for 8.3, 8.2 branch  (Zdenek Kotala <Zdenek.Kotala@Sun.COM>)
Ответы Re: pg_lzcompress patch for 8.3, 8.2 branch  (Zdenek Kotala <Zdenek.Kotala@Sun.COM>)
Список pgsql-patches
Zdenek Kotala <Zdenek.Kotala@Sun.COM> writes:
> I attached backported pg_lzcompress patch which is already in head for version
> 8.2 and 8.3.

> Version 8.1 and prior contains more changes in decompress code and they does not
> contain any check. Shell I backported it as well or it will be better to keep it
> untouched?

AFAICS the only nontrivial patch in pg_lzcompress.c between 7.4 and 8.2
is my cleanup patch here:
http://archives.postgresql.org/pgsql-committers/2006-10/msg00076.php
That's been in the tree long enough that I wouldn't have any hesitation
about back-porting it, so that all the supported versions would have
the same lzcompress code.  About the only reason I can see not to do it
is that conceivably some third-party code somewhere might be calling
pglz_compress directly; in which case an API change in a minor release
would be a problem for them.

On the other hand, I remain unconvinced that this problem is severe
enough to justify much backporting work.  AFAIK we've only seen one
occurence of a problem to date.

Thoughts?

            regards, tom lane

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

Предыдущее
От: Zdenek Kotala
Дата:
Сообщение: pg_lzcompress patch for 8.3, 8.2 branch
Следующее
От: Josh Berkus
Дата:
Сообщение: Re: [HACKERS] WITH RECURSIVE patch V0.1