Re: [PATCH] Incremental backup: add backup profile to base backup - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [PATCH] Incremental backup: add backup profile to base backup
Date
Msg-id 20140818050528.GA8062@eldon.alvh.no-ip.org
Whole thread Raw
In response to [PATCH] Incremental backup: add backup profile to base backup  (Marco Nenciarini <marco.nenciarini@2ndquadrant.it>)
Responses Re: [PATCH] Incremental backup: add backup profile to base backup
List 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 by date:

Previous
From: Fujii Masao
Date:
Subject: Re: [REVIEW] Re: Compression of full-page-writes
Next
From: Michael Paquier
Date:
Subject: Re: pg_receivexlog and replication slots