From 2d4ead6f4bd0e30df15dc0ae654c9ce573f41bed Mon Sep 17 00:00:00 2001 From: Michael Paquier Date: Fri, 6 Mar 2026 14:41:41 +0900 Subject: [PATCH] Fix order of columns in pg_stat_recovery recovery_last_xact_time is listed before current_chunk_start_time in the documentation, the function definition and the view definition, but their order was reversed in the code. Thinko in 01d485b142e4. Mea culpa. Author: Shinya Kato Discussion: https://postgr.es/m/CAOzEurQQ1naKmPJhfE5WOUQjtf5tu08Kw3QCGY5UY=7Rt9fE=w@mail.gmail.com --- src/backend/access/transam/xlogfuncs.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/backend/access/transam/xlogfuncs.c b/src/backend/access/transam/xlogfuncs.c index 7c0e430b690..80007783067 100644 --- a/src/backend/access/transam/xlogfuncs.c +++ b/src/backend/access/transam/xlogfuncs.c @@ -842,14 +842,13 @@ pg_stat_get_recovery(PG_FUNCTION_ARGS) else nulls[5] = true; - if (current_chunk_start_time != 0) - values[6] = TimestampTzGetDatum(current_chunk_start_time); + if (recovery_last_xact_time != 0) + values[6] = TimestampTzGetDatum(recovery_last_xact_time); else nulls[6] = true; - /* recovery_last_xact_time */ - if (recovery_last_xact_time != 0) - values[7] = TimestampTzGetDatum(recovery_last_xact_time); + if (current_chunk_start_time != 0) + values[7] = TimestampTzGetDatum(current_chunk_start_time); else nulls[7] = true;