Re: pg_dump new feature: exporting functions only. Bad or good idea ?

From: Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: ahsan hadi <ahsan(dot)hadi(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Lætitia AVROT <laetitia(dot)avrot(at)gmail(dot)com>
Subject: Re: pg_dump new feature: exporting functions only. Bad or good idea ?
Date: 2021-07-10 11:06:43
Message-ID: a9add6fb-dadd-f2c8-d0d7-97402dc82ba2@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 7/10/21 1:43 AM, Tom Lane wrote:
> Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> writes:
>> The main question I have is whether this should include procedures.
>
> I feel a bit uncomfortable about sticking this sort of limited-purpose
> selectivity mechanism into pg_dump. I'd rather see a general filter
> method that can select object(s) of any type. Pavel was doing some
> work towards that awhile ago, though I think he got frustrated about
> the lack of consensus on details. Which is a problem, but I don't
> think the solution is to accrue more and more independently-designed-
> and-implemented features that each solve some subset of the big problem.
>

I'm not against introducing such general filter mechanism, but why
should it block this patch? I'd understand it the patch was adding a lot
of code, but that's not the case - it's tiny. And we already have
multiple filter options (to pick tables, schemas, extensions, ...).

And if there's no consensus on details of Pavel's patch after multiple
commitfests, how likely is it it'll start moving forward?

regards

--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2021-07-10 11:29:51 Re: bugfix: when the blocksize is 32k, the function page_header of pageinspect returns negative numbers.
Previous Message Dean Rasheed 2021-07-10 11:00:49 Re: Enhanced error message to include hint messages for redundant options error