Re: Add expressions to pg_restore_extended_stats()

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Corey Huinker <corey(dot)huinker(at)gmail(dot)com>
Cc: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Tender Wang <tndrwang(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Tomas Vondra <tomas(at)vondra(dot)me>
Subject: Re: Add expressions to pg_restore_extended_stats()
Date: 2026-03-09 00:07:04
Message-ID: aa4PKL1yBd_7VcsC@paquier.xyz
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Mar 07, 2026 at 01:17:39AM -0500, Corey Huinker wrote:
> Fairly close +307, -603

Thanks for the patches. I have tweaked a couple of comments, and
applied them after merging the whole together. I have also spent the
time checking things by breaking some of the internal logic of the
restore functions, to see if diffs were correctly showing up.

One thing I was wondering is if the flat functions should call the
view, but at the end I guess that it does not matter much as long as
each query is grouped close to the other. There is no need to expose
attrelid in the view, information we do not want to show in the type
of the view anyway.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Mihail Nikalayeu 2026-03-09 00:08:00 Re: [BUG?] check_exclusion_or_unique_constraint false negative
Previous Message Chao Li 2026-03-09 00:04:54 Re: Avoid resource leak (src/bin/pg_dump/pg_dumpall.c)