Tom Lane <t...@sss.pgh.pa.us> writes:
> Dimitri Fontaine <dimi...@2ndquadrant.fr> writes:
>> I've spotted a comment that I think you missed updating. The schema
>> given in the control file is now created in all cases rather than only
>> when the extension is not relocatable, right?
>
> Hm, no, that logic is the same as before no?
Well I had
if (!control->relocatable && control->schema != NULL)
And you have
+ else if (control->schema != NULL)
So you're considering the schema option independently of the relocatable
option, which is ok because you changed those options defaults and
conflicts in the control file parsing. Only the comment needs adjusting.
> I removed it --- it wasn't documented and I didn't see much point anyway
> in a \d command that just duplicates a system view, especially a view
> only usable by superusers.
It used to not be a straight select from system view, and I wanted to
offer something as simple as clicking a check box in pgadmin for people
who want to see what's available and install it. But well, the system
view is good enough now I guess, we're talking about DBA here after all.
Regards,
--
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
