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
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. |
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 |