Re: Allow pageinspect's bt_page_stats function to return a set of rows instead of a single row

From: Andres Freund <andres(at)anarazel(dot)de>
To: Hamid Akhtar <hamid(dot)akhtar(at)percona(dot)com>
Cc: Naeem Akhter <naeem(dot)akhter(at)percona(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Hamid Akhtar <hamid(dot)quddus(at)enterprisedb(dot)com>
Subject: Re: Allow pageinspect's bt_page_stats function to return a set of rows instead of a single row
Date: 2022-12-06 23:16:39
Message-ID: 20221206231639.2croeiamcyzmfaw3@awork3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2022-11-25 02:45:01 +0500, Hamid Akhtar wrote:
> Rebasing the patch to the tip of the master branch.

This doesn't pass tests on cfbot. Looks like possibly some files are missing?

https://api.cirrus-ci.com/v1/artifact/task/4916614353649664/testrun/build/testrun/pageinspect/regress/regression.diffs

diff -U3 /tmp/cirrus-ci-build/contrib/pageinspect/expected/page.out /tmp/cirrus-ci-build/build/testrun/pageinspect/regress/results/page.out
--- /tmp/cirrus-ci-build/contrib/pageinspect/expected/page.out 2022-12-06 20:07:47.691479000 +0000
+++ /tmp/cirrus-ci-build/build/testrun/pageinspect/regress/results/page.out 2022-12-06 20:11:42.955606000 +0000
@@ -1,4 +1,5 @@
CREATE EXTENSION pageinspect;
+ERROR: extension "pageinspect" has no installation script nor update path for version "1.12"
-- Use a temp table so that effects of VACUUM are predictable
CREATE TEMP TABLE test1 (a int, b int);
INSERT INTO test1 VALUES (16777217, 131584);
@@ -6,236 +7,203 @@

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nathan Bossart 2022-12-06 23:27:44 remove extra space from dumped ALTER DEFAULT PRIVILEGES commands
Previous Message Andres Freund 2022-12-06 23:12:30 Re: Using WaitEventSet in the postmaster