Hello,

On 19/11/2025 10:57, Peter Eisentraut wrote:
> Ok, committed that way.

in fce7c73fb (Sort guc_parameters.dat alphabetically by name) two extraneous newlines have been kept in the guc_parameters.dat: before archive_command and before seq_page_cost. I suggest to remove them, since they don't seem to mark any important spots anyway, see the attached patch.

Found by accident after re-sorting guc_parameters.dat in our fork with sort_guc_parameters.pl (also attached just in case). This is a single-use script, not meant to be robust or tidy.

Best regards,
Anton Voloshin
Postgres Professional, The Russian Postgres Company
https://postgrespro.ru
From 590a10d62fee7fcbde26f0d7b93637c3639ee8bd Mon Sep 17 00:00:00 2001
From: Anton Voloshin <[email protected]>
Date: Mon, 22 Jun 2026 11:03:26 +0300
Subject: [PATCH] Remove extraneous newlines from guc_parameters.dat

In fce7c73fb two unnecessary newlines have been kept: before
archive_command and seq_page_cost. Remove them here just to be tidier.
---
 src/backend/utils/misc/guc_parameters.dat | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/backend/utils/misc/guc_parameters.dat b/src/backend/utils/misc/guc_parameters.dat
index afaa058b046..7b1eb6e61bc 100644
--- a/src/backend/utils/misc/guc_parameters.dat
+++ b/src/backend/utils/misc/guc_parameters.dat
@@ -73,7 +73,6 @@
   boot_val => '""',
 },
 
-
 { name => 'archive_command', type => 'string', context => 'PGC_SIGHUP', group => 'WAL_ARCHIVING',
   short_desc => 'Sets the shell command that will be called to archive a WAL file.',
   long_desc => 'An empty string means use "archive_library".',
@@ -2628,7 +2627,6 @@
   boot_val => 'false',
 },
 
-
 { name => 'seq_page_cost', type => 'real', context => 'PGC_USERSET', group => 'QUERY_TUNING_COST',
   short_desc => 'Sets the planner\'s estimate of the cost of a sequentially fetched disk page.',
   flags => 'GUC_EXPLAIN',
-- 
2.54.0

Attachment: sort_guc_parameters.pl
Description: Perl program

Reply via email to