There is an unused function GetOldestWALSendPointer() in walsender.c. Per comment, it was introduced because we may need it in the future for synchronous replication.
Now we have very similar function SyncRepGetOldestSyncRecPtr() in syncrep.c. Which makes me think that GetOldestWALSendPointer() no longer needs to be maintained. So, is it time to remove that unused function?
Seems sensible cleanup to me.
--
Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services