From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Import/export servers |
Date: | 2018-11-20 14:53:22 |
Message-ID: | CA+OCxozGB3p-EN0D1SmUDPdU+RaqNKrVF7nw8jnwT+Khu5huQQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Tue, Nov 20, 2018 at 6:01 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Dave
> On Mon, Nov 19, 2018 at 8:09 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> A number of people have asked for the ability to import and export server
>> definitions. It's also convenient to have such a feature in the container
>> world, where new instances of pgAdmin containers may be created on the fly.
>>
>> Here's a draft patch to allow that, including docs and a basic test.
>>
>> Review feedback please?
>>
>
> Patch looks good to me. Can we have option to provide the servers to
> be exported? It may possible that user don't want to export the local
> servers as it is automatically detected. Can we have GUI(may be dialog)
> for this?
>
This is intentionally not a GUI feature.
I've added a --server flag to allow the user to specify specific servers
(without it, all are dumped), and changed the JSON structure per the
discussion we had earlier to ensure that servers with duplicate names can
be dumped.
Please re-review.
Thanks!
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
import_export_servers_v2.diff | application/octet-stream | 24.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-11-20 16:13:47 | Re: [pgAdmin4][Patch]: RM #3354 Debugger does not start if function has input vars of type array |
Previous Message | Dave Page | 2018-11-20 13:50:47 | Re: Import/export servers |