Skip site navigation (1) Skip section navigation (2)

Re: Extensions support for pg_dump, patch v27

From: Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>
To: Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>
Cc: PostgreSQL-Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Extensions support for pg_dump, patch v27
Date: 2011-01-31 03:42:47
Message-ID: AANLkTins+Pq6zB3r=PW5YXbkSU18Mn3wn-Obq+MT9fbC@mail.gmail.com (view raw or flat)
Thread:
Lists: pgsql-hackers
On Fri, Jan 28, 2011 at 18:03, Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr> wrote:
> After review, I included all your proposed changes, thanks a lot!
> Please find attached a new version of the patch, v29,

Additional questions and discussions:

* "relocatable" and "schema" seems to be duplicated options.
We could treat an extension is relocatable when schema is not
specified instead of relocatable option. Or, If we keep "schema"
option, it would be used as the default schema to be installed
when WITH SCHEMA is not specified.

* "version" field in pg_available_extension might mislead when
a newer version of an extension is available but an older version
is installed. How about returning installed version for "installed"
field instead of booleans? The field will be NULLs if not installed.

* I want to remove O(n^2) behavior in pg_extensions(). It scans
pg_extension catalog to return whether the extension is installed,
but it would be better to change the function to return just whole
extensions and JOIN with pg_extension in pg_available_extensions.
(it's the same technique used in pg_stat_replication)

-- 
Itagaki Takahiro

In response to

Responses

pgsql-hackers by date

Next:From: Fujii MasaoDate: 2011-01-31 03:44:30
Subject: Re: pg_ctl failover Re: Latches, signals, and waiting
Previous:From: Hitoshi HaradaDate: 2011-01-31 03:42:28
Subject: Re: Add ENCODING option to COPY

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group