Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-26 Thread Tom Lane
Bharath Rupireddy writes: > On Sat, Mar 26, 2022 at 10:57 PM Andres Freund wrote: >> -1. I think it's a perfectly reasonable function to have, it doesn't cause >> architectural / maintenance issues to have it and there's several plausible >> future uses for it (moving fsyncing of received WAL to

Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-26 Thread Bharath Rupireddy
On Sat, Mar 26, 2022 at 10:57 PM Andres Freund wrote: > > Hi, > > On 2022-03-26 10:51:15 +0530, Bharath Rupireddy wrote: > > The function GetWalRcvWriteRecPtr isn't being used anywhere, however > > pg_atomic_read_u64(&walrcv->writtenUpto); (reading writtenUpto without > > spinlock) is being used d

Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-26 Thread Andres Freund
Hi, On 2022-03-26 10:51:15 +0530, Bharath Rupireddy wrote: > The function GetWalRcvWriteRecPtr isn't being used anywhere, however > pg_atomic_read_u64(&walrcv->writtenUpto); (reading writtenUpto without > spinlock) is being used directly in pg_stat_get_wal_receiver > walreceiver.c. We either make

Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-26 Thread Tom Lane
Julien Rouhaud writes: > On Sat, Mar 26, 2022 at 02:52:29PM +0900, Michael Paquier wrote: >> This could be used by some external module, no? > Maybe, but WalRcv is exposed so if an external module needs it it could still > maintain its own version of GetWalRcvWriteRecPtr. We'd need to mark WalRc

Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-26 Thread Bharath Rupireddy
On Sat, Mar 26, 2022 at 12:55 PM Julien Rouhaud wrote: > > On Sat, Mar 26, 2022 at 02:52:29PM +0900, Michael Paquier wrote: > > On Sat, Mar 26, 2022 at 10:51:15AM +0530, Bharath Rupireddy wrote: > > > The function GetWalRcvWriteRecPtr isn't being used anywhere, however > > > pg_atomic_read_u64(&wa

Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-26 Thread Julien Rouhaud
On Sat, Mar 26, 2022 at 02:52:29PM +0900, Michael Paquier wrote: > On Sat, Mar 26, 2022 at 10:51:15AM +0530, Bharath Rupireddy wrote: > > The function GetWalRcvWriteRecPtr isn't being used anywhere, however > > pg_atomic_read_u64(&walrcv->writtenUpto); (reading writtenUpto without > > spinlock) is

Re: Remove an unused function GetWalRcvWriteRecPtr

2022-03-25 Thread Michael Paquier
On Sat, Mar 26, 2022 at 10:51:15AM +0530, Bharath Rupireddy wrote: > The function GetWalRcvWriteRecPtr isn't being used anywhere, however > pg_atomic_read_u64(&walrcv->writtenUpto); (reading writtenUpto without > spinlock) is being used directly in pg_stat_get_wal_receiver > walreceiver.c. We eithe

Remove an unused function GetWalRcvWriteRecPtr

2022-03-25 Thread Bharath Rupireddy
Hi, The function GetWalRcvWriteRecPtr isn't being used anywhere, however pg_atomic_read_u64(&walrcv->writtenUpto); (reading writtenUpto without spinlock) is being used directly in pg_stat_get_wal_receiver walreceiver.c. We either make use of the function instead of pg_atomic_read_u64(&walrcv->writ