Re: lz4 --rm on Ubuntu 18.04 (Add LZ4 compression to pg_dump)

Поиск
Список
Период
Сортировка
От Tomas Vondra
Тема Re: lz4 --rm on Ubuntu 18.04 (Add LZ4 compression to pg_dump)
Дата
Msg-id ef0cbd4e-0e52-e634-da52-1609e27e939e@enterprisedb.com
обсуждение исходный текст
Ответ на Re: lz4 --rm on Ubuntu 18.04 (Add LZ4 compression to pg_dump)  (Christoph Berg <myon@debian.org>)
Список pgsql-hackers
On 3/14/23 11:34, Christoph Berg wrote:
> Re: Tomas Vondra
>> and I don't think there's a good place to inject the 'rm' so I ended up
>> adding a 'cleanup_cmd' right after 'compress_cmd'. But it seems a bit
>> strange / hacky. Maybe there's a better way?
> 
> Does the file need to be removed at all? Could we leave it there and
> have "make clean" remove it?
> 

I don't think that'd work, because of the automatic "discovery" where we
check if a file exists, and if not we try to append .gz and .lz4. So if
you leave the .toc, we'd not find the .lz4, making the test useless ...


regards

-- 
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



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

Предыдущее
От: Nazir Bilal Yavuz
Дата:
Сообщение: Re: meson: Non-feature feature options
Следующее
От: Justin Pryzby
Дата:
Сообщение: Re: Progress report of CREATE INDEX for nested partitioned tables