Fix debug message output when connecting to a logical slot.

Previously the message erroneously printed the same LSN twice as the
assignment to the start_lsn variable was before the message. Correct
that.

Reported-By: Marko Tiikkaja
Author: Marko Tiikkaja
Backpatch: 9.5, where logical decoding was introduced

diff --git a/src/backend/replication/logical/logical.c b/src/backend/replication/logical/logical.c
index 824bc915b1d1b6160a9f5af1d1c47b86db76669b..7643add6d2cc3f63c9ec7b72b5c0dbe340ef6929 100644 (file)
--- a/src/backend/replication/logical/logical.c
+++ b/src/backend/replication/logical/logical.c
@@ -406,11 +406,12 @@ CreateDecodingContext(XLogRecPtr start_lsn,
                 * decoding. Clients have to be able to do that to support synchronous
                 * replication.
                 */
-               start_lsn = slot->data.confirmed_flush;
                elog(DEBUG1, "cannot stream from %X/%X, minimum is %X/%X, forwarding",
                         (uint32) (start_lsn >> 32), (uint32) start_lsn,
                         (uint32) (slot->data.confirmed_flush >> 32),
                         (uint32) slot->data.confirmed_flush);
+
+               start_lsn = slot->data.confirmed_flush;
        }
 
        ctx = StartupDecodingContext(output_plugin_options,