Re: [pgAdmin4][RM#3140] Add service parameter
От | Murtuza Zabuawala |
---|---|
Тема | Re: [pgAdmin4][RM#3140] Add service parameter |
Дата | |
Msg-id | CAKKotZRvWmbGLfUWG2S4-5TAsV2TT0MoByTLYys1y2xFtkvBmg@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [pgAdmin4][RM#3140] Add service parameter (Ashesh Vashi <ashesh.vashi@enterprisedb.com>) |
Ответы |
Re: [pgAdmin4][RM#3140] Add service parameter
(Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Re: [pgAdmin4][RM#3140] Add service parameter (Dave Page <dpage@pgadmin.org>) |
Список | pgadmin-hackers |
Hi Ashesh,
I haven't implemented that intentionally because Khushboo is working on the same for SSL and our code will conflict, So once Khushboo's patch gets committed, I'll make changes for Service file as well.
--
Regards,
On Tue, Mar 13, 2018 at 9:01 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Murtuza/Dave,I have to reviewed/seen the patch yet.But - I have a question.Have we used the service file in the external tools for backup, restore, and import/export functionalities?If not - we should fix that asap.We had missed that during SSL support, and now - we're fixing that.On Tue, Mar 13, 2018 at 6:18 AM, Dave Page <dpage@pgadmin.org> wrote:HiOn Mon, Mar 12, 2018 at 5:18 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hi Dave and Murtuza,Regarding this patch we refactored the Javascript code so that is lives in a different file and added some tests.Also we found an issue with karma-jasmine that does not allow us to use jasmine 3.1 yet. You can find attached a patch that reverts that commit.Sounds good, but neither patch will apply (in fact, the Jasmine one looks entirely backwards). One of the error messages was changed in Murtuza's patch, and wasn't reflected in your update for example.Can you rebase please?Thanks.ThanksVictoria && JoaoOn Mon, Mar 12, 2018 at 4:46 PM Dave Page <dpage@pgadmin.org> wrote:Thanks, patch applied!On Mon, Mar 12, 2018 at 3:31 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi Dave,PFA updated patch.--Regards,On Fri, Mar 9, 2018 at 9:29 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi Dave,I'll change the name and send you updated patch.On Fri, Mar 9, 2018 at 9:25 PM, Dave Page <dpage@pgadmin.org> wrote:HIOn Fri, Mar 9, 2018 at 11:47 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi,PFA patch to add service parameter in server dialog.- Docs updated- Test case added for Service ID parameterPlease note,I have extracted Connection class and Server manager class from our own custom Psycopg2 driver module.Patch also covers RM#3120This patch seems a little confused. The "Service" and "Service ID" fields from pgAdmin 3 are very different things. The Redmine ticket seems to be asking for the Service field (the pg_service.conf service name), *not* Service ID (the operating system's service ID, used to start/stop the database server service).--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
В списке pgadmin-hackers по дате отправления:
Предыдущее
От: Dave PageДата:
Сообщение: Re: [pgAdmin4][Patch]: RM #2963 - Backup database, Restore database and Maintenance Database failed for é object.