From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Nicolas Thauvin <nicolas(dot)thauvin(at)dalibo(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Foreign tables privileges not shown in information_schema.table_privileges |
Date: | 2017-08-14 19:35:39 |
Message-ID: | 83297201-864a-0430-8e92-111d9dba03b8@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 8/11/17 04:52, Ashutosh Bapat wrote:
> On Thu, Aug 10, 2017 at 6:30 PM, Nicolas Thauvin
> <nicolas(dot)thauvin(at)dalibo(dot)com> wrote:
>> Hello,
>>
>> The information_schema.table_privileges view filters on regular tables
>> and views. Foreign tables are not shown in this view but they are in
>> other views of the information_schema like tables or column_privileges.
>>
>> Is it intentional? A patch is attached if not.
>
> The line was first added by 596652d6 and updated by 262e821d to
> include partitioned tables. Looks like we have forgot to add tables
> added in between i.e. foreign tables and materialized views.
> column_privileges doesn't have materialized views. Attached patch adds
> materialized views to column_privileges view along with your changes.
I see several neighboring issues here:
- Foreign tables privileges not shown in
information_schema.table_privileges -- That is an omission that should
be fixed.
- information_schema.tables shows table_type 'FOREIGN TABLE', but it
should be 'FOREIGN' per SQL standard.
- Materialized views not included. I think that is an intentional
omission. It's valid to reconsider, but it would be to be a separate
discussion.
I think I would fix #1 and #2 with back patches but no catversion change.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2017-08-14 19:48:51 | Re: Add Roman numeral conversion to to_number |
Previous Message | Chris Travers | 2017-08-14 19:27:26 | Re: Orphaned files in base/[oid] |