Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-11-13 Thread Andres Freund
On 2014-11-14 00:04:52 +0200, Heikki Linnakangas wrote: > On 11/11/2014 06:55 PM, Andres Freund wrote: > >On 2014-11-03 21:58:51 +0200, Heikki Linnakangas wrote: > >>PS. I find the name "ReplicationSlotOnDiskDynamicSize" confusing, as it is > >>in fact a fixed size struct. I gather it's expected th

Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-11-13 Thread Heikki Linnakangas
On 11/11/2014 06:55 PM, Andres Freund wrote: On 2014-11-03 21:58:51 +0200, Heikki Linnakangas wrote: PS. I find the name "ReplicationSlotOnDiskDynamicSize" confusing, as it is in fact a fixed size struct. I gather it's expected that the size of that part might change across versions, but by that

Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-11-11 Thread Andres Freund
On 2014-11-03 21:58:51 +0200, Heikki Linnakangas wrote: > PS. I find the name "ReplicationSlotOnDiskDynamicSize" confusing, as it is > in fact a fixed size struct. I gather it's expected that the size of that > part might change across versions, but by that definition nothing is > constant. Well,

Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-11-03 Thread Heikki Linnakangas
On 10/31/2014 03:46 PM, Andres Freund wrote: On 2014-10-27 09:30:33 -0400, Tom Lane wrote: Andres Freund writes: On 2014-10-27 12:51:44 +0200, Heikki Linnakangas wrote: replication/slot.c and replication/logical/snapbuild.c use a CRC on the physical slot and snapshot files. It uses the same a

Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-10-31 Thread Andres Freund
On 2014-10-27 09:30:33 -0400, Tom Lane wrote: > Andres Freund writes: > > On 2014-10-27 12:51:44 +0200, Heikki Linnakangas wrote: > >> replication/slot.c and replication/logical/snapbuild.c use a CRC on the > >> physical slot and snapshot files. It uses the same algorithm as used e.g. > >> for the

Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-10-27 Thread Tom Lane
Andres Freund writes: > On 2014-10-27 12:51:44 +0200, Heikki Linnakangas wrote: >> replication/slot.c and replication/logical/snapbuild.c use a CRC on the >> physical slot and snapshot files. It uses the same algorithm as used e.g. >> for the WAL. However, they are not doing the finalization step,

Re: [HACKERS] Missing FIN_CRC32 calls in logical replication code

2014-10-27 Thread Andres Freund
On 2014-10-27 12:51:44 +0200, Heikki Linnakangas wrote: > replication/slot.c and replication/logical/snapbuild.c use a CRC on the > physical slot and snapshot files. It uses the same algorithm as used e.g. > for the WAL. However, they are not doing the finalization step, FIN_CRC32() > on the calcul