Re: [PATCH] Incremental backup: add backup profile to base backup

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: [PATCH] Incremental backup: add backup profile to base backup
Дата
Msg-id 20140818050528.GA8062@eldon.alvh.no-ip.org
обсуждение исходный текст
Ответ на [PATCH] Incremental backup: add backup profile to base backup  (Marco Nenciarini <marco.nenciarini@2ndquadrant.it>)
Ответы Re: [PATCH] Incremental backup: add backup profile to base backup
Список pgsql-hackers
Marco Nenciarini wrote:

> To calculate the md5 checksum I've used the md5 code present in pgcrypto
> contrib as the code in src/include/libpq/md5.h is not suitable for large
> files. Since a core feature cannot depend on a piece of contrib, I've
> moved the files
> 
> contrib/pgcrypto/md5.c
> contrib/pgcrypto/md5.h
> 
> to
> 
> src/backend/utils/hash/md5.c
> src/include/utils/md5.h
> 
> changing the pgcrypto extension to use them.

We already have the FNV checksum implementation in the backend -- can't
we use that one for this and avoid messing with MD5?

(I don't think we're looking for a cryptographic hash here.  Am I wrong?)

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services



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

Предыдущее
От: Fujii Masao
Дата:
Сообщение: Re: [REVIEW] Re: Compression of full-page-writes
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: pg_receivexlog and replication slots