Date: Thu, 12 Sep 2024 21:12:29 -0500
From: Nathan Bossart <nathandbossart@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Matthias van de Meent <boekewurm+postgres@gmail.com>,
	Robert Haas <robertmhaas@gmail.com>,
	Ayush Vatsa <ayushvatsa1810@gmail.com>,
	PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: Pgstattuple on Sequences: Seeking Community Feedback on
 Potential Patch
Message-ID: <ZuOfjemYJfjpio2u@nathan>
References: 
 <CAEze2WiNB=wjrtQGERk2mmkpOZevGCfTT7ei6guJsFG4VRmacA@mail.gmail.com>
 <Ztd0VcV2PQJf5jUJ@nathan>
 <ZuH_PB1S3JuWvNPq@nathan>
 <CAEze2Wh_KkMthGmGiW+v9hFN90L066Vt3VQdEFmwmi03CgFRkA@mail.gmail.com>
 <ZuJPlHWN6HOO9AYD@paquier.xyz>
 <ZuNfgqVMc2O9XQN6@nathan>
 <ZuOKOut5hhDlf_bP@nathan>
 <ZuOLqG5zXKEB9ICg@paquier.xyz>
 <ZuON2ERKm9nvkOYo@nathan>
 <ZuOYcb9hYfMNaHRD@nathan>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="ccDEccsyTIPDjiIw"
Content-Disposition: inline
In-Reply-To: <ZuOYcb9hYfMNaHRD@nathan>
Archived-At: <https://www.postgresql.org/message-id/ZuOfjemYJfjpio2u%40nathan>
Precedence: bulk


--ccDEccsyTIPDjiIw
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Thu, Sep 12, 2024 at 08:42:09PM -0500, Nathan Bossart wrote:
> Oh, the answer seems to be commits aeaaf52 and 47a22dc.  In short, we can't
> use a temporary sequence in this test for versions older than v15.

Here's a patch to make the sequence permanent and to make the output of
tuple_data_split() not depend on endianness.

-- 
nathan

--ccDEccsyTIPDjiIw
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="fix_pageinspect_test.patch"

diff --git a/contrib/pageinspect/expected/page.out b/contrib/pageinspect/expected/page.out
index 04fd9dee4b..3fd3869c82 100644
--- a/contrib/pageinspect/expected/page.out
+++ b/contrib/pageinspect/expected/page.out
@@ -240,11 +240,12 @@ SELECT page_checksum(decode(repeat('00', :block_size), 'hex'), 1);
 (1 row)
 
 -- tests for sequences
-create temporary sequence test_sequence;
+create sequence test_sequence start 72057594037927937;
 select tuple_data_split('test_sequence'::regclass, t_data, t_infomask, t_infomask2, t_bits)
   from heap_page_items(get_raw_page('test_sequence', 0));
                    tuple_data_split                    
 -------------------------------------------------------
- {"\\x0100000000000000","\\x0000000000000000","\\x00"}
+ {"\\x0100000000000001","\\x0000000000000000","\\x00"}
 (1 row)
 
+drop sequence test_sequence;
diff --git a/contrib/pageinspect/sql/page.sql b/contrib/pageinspect/sql/page.sql
index 59784fc7cc..346e4ee142 100644
--- a/contrib/pageinspect/sql/page.sql
+++ b/contrib/pageinspect/sql/page.sql
@@ -100,6 +100,7 @@ SELECT page_header(decode(repeat('00', :block_size), 'hex'));
 SELECT page_checksum(decode(repeat('00', :block_size), 'hex'), 1);
 
 -- tests for sequences
-create temporary sequence test_sequence;
+create sequence test_sequence start 72057594037927937;
 select tuple_data_split('test_sequence'::regclass, t_data, t_infomask, t_infomask2, t_bits)
   from heap_page_items(get_raw_page('test_sequence', 0));
+drop sequence test_sequence;

--ccDEccsyTIPDjiIw--