Re: Import/export servers

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

In response to

Responses

Browse pgadmin-hackers by date

  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