Re: "WIP: Data at rest encryption" patch and, 2 phase commit. - Mailing list pgsql-hackers

From Toshi Harada
Subject Re: "WIP: Data at rest encryption" patch and, 2 phase commit.
Date
Msg-id 201807270708.w6R78pgs001650@ccmail04.silk.ntt-tx.co.jp
Whole thread Raw
In response to Re: "WIP: Data at rest encryption" patch and, 2 phase commit.  (Antonin Houska <ah@cybertec.at>)
List pgsql-hackers
Hi.

Applying https://www.postgresql.org/message-id/11678.1532519255%40localhost patch, 
the problem of pg_create_logical_replication_slot () and the 2PC problem were solved.

Thanks.

Antonin Houska <ah@cybertec.at> wrote:
> Toshi Harada <harada.toshi@po.ntt-tx.co.jp> wrote:
> 
> > Hi.
> > 
> > I applied the patch "WIP: Data at rest encryption" to PostgreSQL 11 - beta 2 and I'm working on it.
> > 
> > When this patch is applied, the following problem occurs.
> > 
> > * An error occurs when CHECKPOINT is executed during two-phase commit.
> > * After an error occurs, if you stop PostgreSQL, it will never start again.
> > 
> > (1) First, execute PREPARE TRANSACTION.
> > 
> > postgres=# BEGIN;
> > BEGIN
> > postgres=# PREPARE TRANSACTION 'foo';
> > PREPARE TRANSACTION
> > postgres=#
> > 
> > (2) Execute the CHECKPOINT command from another terminal.
> >     CHEKPOINT command fails.
> > 
> > postgres=# CHECKPOINT;
> > ERROR:  checkpoint request failed
> > HINT:  Consult recent messages in the server log for details.
> > postgres=#
> 
> The patch version I posted in
> 
> https://www.postgresql.org/message-id/11678.1532519255%40localhost
> 
> fixes an issue (unitialized pointer) that caused failure here, but it was
> SEGFAULT rather than ERROR. And the scope of the bug was broader than just
> CHECKPOINT.
> 
> Can you please test it again with the new version of the patch?
> 
> Anyway, thanks for your reports!
> 
> 
> -- 
> Antonin Houska
> Cybertec Schonig & Schonig GmbH
> Grohrmuhlgasse 26, A-2700 Wiener Neustadt
> Web: https://www.cybertec-postgresql.com
> 




pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: print_path is missing GatherMerge and CustomScan support
Next
From: Amit Langote
Date:
Subject: Re: Speeding up INSERTs and UPDATEs to partitioned tables