Re: WIP: Data at rest encryption - Mailing list pgsql-hackers

From Shawn Wang
Subject Re: WIP: Data at rest encryption
Date
Msg-id 156472805641.1401.3119816532289012709.pgcf@coridan.postgresql.org
Whole thread Raw
In response to Re: "WIP: Data at rest encryption" patch and, PostgreSQL 11-beta3  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: WIP: Data at rest encryption  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers
The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, failed
Spec compliant:           tested, failed
Documentation:            not tested

Hi Antonin,
It is very glad to see the new patch. I used the public patches a long time ago.
I did some tests like the stream replication, much data running, temporary files encryption.
I found that there is an issue in the src/backend/storage/file/encryption.c. You should put block_size =
EVP_CIPHER_CTX_block_size(ctx);under the #ifdef USE_ASSERT_CHECKING.
 
There is some problem to merge your patches to the latest kernel in the pg_ctl.c.

Regards,

--
Shawn Wang

pgsql-hackers by date:

Previous
From: Fabien COELHO
Date:
Subject: Re: pgbench - implement strict TPC-B benchmark
Next
From: Adam Lee
Date:
Subject: Re: Memory-Bounded Hash Aggregation