Consistently set all fields in pg_stat_replication to null instead of 0

Previously the "sent" field would be set to 0 and all other xlog
pointers be set to NULL if there were no valid values (such as when
in a backup sending walsender).

diff --git a/src/backend/replication/walsender.c b/src/backend/replication/walsender.c
index c13567298c2f9de7c295e2a5a29e852ce41fb541..7b1b7f14922a88386ef2a5fef2bdeb4066aace2b 100644 (file)
--- a/src/backend/replication/walsender.c
+++ b/src/backend/replication/walsender.c
@@ -2822,6 +2822,9 @@ pg_stat_get_wal_senders(PG_FUNCTION_ARGS)
                else
                {
                        values[1] = CStringGetTextDatum(WalSndGetStateString(state));
+
+                       if (XLogRecPtrIsInvalid(sentPtr))
+                               nulls[2] = true;
                        values[2] = LSNGetDatum(sentPtr);
 
                        if (XLogRecPtrIsInvalid(write))