| From: | Adam Brightwell <adam(dot)brightwell(at)crunchydatasolutions(dot)com> |
|---|---|
| To: | Stephen Frost <sfrost(at)snowman(dot)net> |
| Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Petr Jelinek <petr(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Additional role attributes && superuser review |
| Date: | 2014-11-18 21:58:46 |
| Message-ID: | CAKRt6CTgJdeGFqXevrp-DizaeHmg8gNVqu8n5T=ix3JAvpwwDQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
All,
> Currently, I am using int32 simply because int64 is causing some issues.
> The issue is that genbki.pl is not able to associate it with the int8
> type as defined in pg_type.h. Therefore Schema_pg_authid in schemapg.h
> isn't defined correctly. I've been digging and scratching my head on this
> one but I have reached a point where I think it would be better just to ask.
>
Attached is a quite trivial patch that addresses the int64 (C) to int8
(SQL) mapping issue.
Further digging revealed that Catalog.pm wasn't accounting for int64
(thanks Stephen). Would it be better to include this change as a separate
patch (as attached) or would it be preferable to include with a larger role
attribute bitmask patch?
Thanks,
Adam
--
Adam Brightwell - adam(dot)brightwell(at)crunchydatasolutions(dot)com
Database Engineer - www.crunchydatasolutions.com
| Attachment | Content-Type | Size |
|---|---|---|
| int64_catalog_atttype.patch | text/x-patch | 465 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Geoghegan | 2014-11-18 22:01:13 | Re: GIN pageinspect functions |
| Previous Message | Tom Lane | 2014-11-18 21:42:49 | Use of recent Russian TZ changes in regression tests |