Re: [COMMITTERS] pgsql: Include foreign tables in information_schema.table_privileges

From: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Include foreign tables in information_schema.table_privileges
Date: 2017-08-17 00:22:27
Message-ID: 88bfa467-fe33-2a91-bd80-b269dd44095b@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On 8/16/17 07:27, Robert Haas wrote:
> On Tue, Aug 15, 2017 at 7:41 PM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
>> Include foreign tables in information_schema.table_privileges
>>
>> This appears to have been an omission in the original commit
>> 0d692a0dc9f. All related information_schema views already include
>> foreign tables.
>>
>> Reported-by: Nicolas Thauvin <nicolas(dot)thauvin(at)dalibo(dot)com>
>
> I would have been disinclined to back-patch this. Somebody could be
> depending on the current behavior, and now we'll end up with two kinds
> of clusters, one where initdb was done before these changes and
> another where it was done afterward. So nobody will be able to count
> on this behavior one way or the other.

I understand that. But this is an additive change, so to speak, so
whatever behavior someone was relying on should continue to work.

I did not backpatch 9b5140fb503eb50634cd7e080d41f4d9af41e0a6 because
that is a behavior-breaking change.

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2017-08-17 14:38:15 pgsql: doc: Fix table column count
Previous Message Peter Eisentraut 2017-08-16 23:50:39 Re: Re: [COMMITTERS] pgsql: Don't force-assign transaction id when exporting a snapshot.

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2017-08-17 00:45:42 Re: Fix number skipping in to_number
Previous Message Craig Ringer 2017-08-17 00:20:54 Re: [PATCH] pageinspect function to decode infomasks