Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lFZNF-0004jR-3F for pgadmin-hackers@arkaria.postgresql.org; Fri, 26 Feb 2021 09:25:25 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1lFZND-0000XY-Tz for pgadmin-hackers@arkaria.postgresql.org; Fri, 26 Feb 2021 09:25:23 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lFZND-0000XR-JU for pgadmin-hackers@lists.postgresql.org; Fri, 26 Feb 2021 09:25:23 +0000 Received: from mail-io1-xd2a.google.com ([2607:f8b0:4864:20::d2a]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1lFZNA-000456-9s for pgadmin-hackers@lists.postgresql.org; Fri, 26 Feb 2021 09:25:23 +0000 Received: by mail-io1-xd2a.google.com with SMTP id g27so7615042iox.2 for ; Fri, 26 Feb 2021 01:25:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=enterprisedb-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=EMGByb6t/rKktwXc4i7iIqZC7qgK1zvREh6OSObYcXY=; b=TUMUfURYfxGA2NlIz63I4UtsIJbAGG6+3QYx3WvLd+HpfwNL+dL4FABu7ucAa/gmns gnrS8Wy72MyKRIGBLyzeJzZGkRJDnusBSvYrCMgwyhnVFYx6Xb7AgbyJmvkFNgarf3Xl CFOW+ge3KXMTOHLXA6t7qWPcA5/yhupyI5CvWEkzmsku40JmAMm6pLQex7BI55wJL/Oh tG2dsdc0Fjij95BkCIik6WEk60Lff3HBwg42rpn+JQwhQYiALWm55Ak+JlB9rwG8PjAr n36KMcvVQGKQZ9V8OC95ROvVfGbhFHemd7aDr54wk43JbtQzhqB6wUwa/nUu39INNU9u wD/g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=EMGByb6t/rKktwXc4i7iIqZC7qgK1zvREh6OSObYcXY=; b=COQbiiZDOjXhO9u+/1lmrkQUSRfODG7ilZk4ct2w/r2YKA1c+zWtoJIbTn23RPGBXW 11adXGVhHkjYCrtIST/FKnIFastW6zOeiBEOXfWop6QdAecQ2pRotYbOlZiQsnOfFvPR BrAWlqxIVT2sQ1/yswtG0WcIaS5gtDfsn4DI8fHsui4X6lEpqWilaoDwHoHCnNeBut9V 9W44LvemObVXqjFK47UfTPYIrcxaBjDgupMr+ASdobxW49vqiCqrf+/sKdMjGNg0eu+t jtsjX3EeSPettTb+vVkJOvbcsRXRkPc5akvmnGdIIrOaC2zBK7jMaNEjUZARADLPgXom UwzQ== X-Gm-Message-State: AOAM532PxGA3hIAvstWpoZhtTINxeqzAtc0YNFcmhGGVRHPzkJwjbMh6 9ykedEcMm5x0URBayvIsdiMN4GrMrGE/FROVuGkE1tgoUyabCHUYE/6V46l1V2Nxqwcyv8KtaEZ sh7m+j79ZsuypiXcDdLcSiKDqF4EIqRZxT6g2ZJZdZ82r4yl1EBci/p3UAj6/EnS8NiG8iGu9RG kssqtkk8y8gZVX9ZaHrxVfZnh5uisoaPPzJm5kw+ihOTXHs86ciVXy6zc0UnxUnQ0EUdb3JJ0Oz w== X-Google-Smtp-Source: ABdhPJwGTvT85G8jHQZ5XrN8nLBeek6rF+96lWTk/ElMG99NyarOClVXpd3ErBczBBzztedgoqL7c5ArRtEQgcZShHo= X-Received: by 2002:a02:ec5:: with SMTP id 188mr2057273jae.20.1614331517863; Fri, 26 Feb 2021 01:25:17 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Akshay Joshi Date: Fri, 26 Feb 2021 14:55:07 +0530 Message-ID: Subject: Re: Feature 6270: Allow for --load-servers to replace current server list To: Alessandro De Maria Cc: pgadmin-hackers Content-Type: multipart/alternative; boundary="000000000000ac297905bc39d84d" X-CLOUD-SEC-AV-Info: enterprisedb,google_mail,monitor X-CLOUD-SEC-AV-Sent: true X-Gm-Spam: 0 X-Gm-Phishy: 0 List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --000000000000ac297905bc39d84d Content-Type: text/plain; charset="UTF-8" Hi Alessandro Patch looks good to me. Please fix the following issues: - Fix the PEP8 issue. - Documentation needs to be updated with this new flag. Please update "docs-> en_Us -> import_export_servers.rst" file. On Thu, Feb 25, 2021 at 10:06 PM Alessandro De Maria < alessandro.demaria@gmail.com> wrote: > Hello, > > I have created a patch for the issue I raised earlier: > > https://redmine.postgresql.org/issues/6270 > > I apologise I have never sent git patches before, and I could not find any > specific documentation of how you expect to receive contributions. > > Regards > Alessandro > > -- > Alessandro De Maria > alessandro.demaria@gmail.com > -- *Thanks & Regards* *Akshay Joshi* *pgAdmin Hacker | Principal Software Architect* *EDB Postgres * *Mobile: +91 976-788-8246* --000000000000ac297905bc39d84d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi=C2=A0Alessandro

Patch looks good to = me. Please fix the following issues:
  • Fix the PEP8 issue.<= /li>
  • Documentation needs to be updated with this new flag. Please update= "docs-> en_Us -> import_export_servers.rst" file.=C2=A0

On Thu, Feb 25, 2021 at 10:06 PM Alessandro De Maria <alessandro.demaria@gmail.com= > wrote:
Hello,

I have created a patch for the issue= I raised earlier:

https://redmine.postgresql.org/issues/6270

I apologise I have never sent git patches before, and = I could not find any specific documentation of how you expect to receive co= ntributions.

Regards
Alessandro

--


--
Thanks & Regards
Akshay Joshi
pgAdmi= n Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

--000000000000ac297905bc39d84d--