From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Anthony Emengo <aemengo(at)pivotal(dot)io>, Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 |
Date: | 2018-05-22 09:01:13 |
Message-ID: | CANxoLDfUFUekR3SXgYSY=+98LEcgg5s-Bk5Yfs9xjLbd3BvrGQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave
On Tue, May 22, 2018 at 2:02 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>
> On Tue, May 22, 2018 at 9:13 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Tue, May 22, 2018 at 7:07 AM, Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> As per suggestion by Dave, I have modified the logic and now
>>> notifications are popped up in alertify dialog(refer Notify_Messages.png)
>>> as and when received on that session where "LISTEN" is executed. Attached
>>> is the modified patch, please review it.
>>>
>>> To test this feature following steps need to perform:
>>>
>>> - Apply the patch.
>>> - Run pgAdmin4
>>> - Connect to any database server and open query tool.
>>> - Execute 'LISTEN foo;' command.
>>> - Open another query tool window and execute 'NOTIFY foo'. (This is
>>> without payload).
>>> - Execute 'select pg_notify('foo', 'Hello')' query (with payload).
>>> - Go to the query tool window from where 'LISTEN' was executed and
>>> run any other query.
>>>
>>> I think there was a small misunderstanding here - I was suggesting that
>> each notification be displayed in an Alertify notification, e.g. using
>> alertify.message('A notification of FOO was received with payload
>> '1234'...')
>>
>
If there are too many notifications then it's annoying for user to
popped up N number of alertify dialogs. Notification is only receives when
any other query execute on the session where "LISTEN" command executes. So
for example I have NOTIFY 10 times from different sessions and execute any
other query on the session("LISTEN" one), 10 alertify dialog will be popped
up.
>
> And it failed tests: https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/
> pipelines/pgadmin-patch/jobs/run-tests/builds/85 :-(
>
> Again it's timeout issue, not able to reproduce on my machine will
look into it. Maybe will have to add webDriverWait.
>
>>
>>
>>>
>>> -
>>>
>>>
>>>
>>>
>>> On Mon, May 21, 2018 at 1:36 PM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Dave
>>>>
>>>> On Fri, May 18, 2018 at 4:56 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Fri, May 18, 2018 at 12:11 PM, Akshay Joshi <
>>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi Dave
>>>>>>
>>>>>> On Fri, May 18, 2018 at 3:58 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>>
>>>>>>> Hi
>>>>>>>
>>>>>>> On Wed, May 16, 2018 at 2:51 PM, Anthony Emengo <aemengo(at)pivotal(dot)io>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Hey,
>>>>>>>>
>>>>>>>> The code looks great! The tests all passed as well.
>>>>>>>>
>>>>>>>
>>>>>>> Agreed - however, unless you check the Messages panel, you're not
>>>>>>> likely to see that a message was received.
>>>>>>>
>>>>>>> Can we also show each message in an alertify panel?
>>>>>>>
>>>>>>
>>>>>> We need to change the design I guess, as we are currently send
>>>>>> this as part of Messages. We will have to send this separately and show it
>>>>>> in the alertify panel.
>>>>>>
>>>>>
>>>>> Yeah. Unfortunately I think notifications need to be more "active"
>>>>> than the messages.
>>>>>
>>>>
>>>> I am working on the above. Can we add one preferences setting to
>>>> "ON/OFF" this alertify panel ?
>>>>
>>>>>
>>>>> --
>>>>> Dave Page
>>>>> Blog: http://pgsnake.blogspot.com
>>>>> Twitter: @pgsnake
>>>>>
>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> *Akshay Joshi*
>>>>
>>>> *Sr. Software Architect *
>>>>
>>>>
>>>>
>>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>>
>>>
>>>
>>>
>>> --
>>> *Akshay Joshi*
>>>
>>> *Sr. Software Architect *
>>>
>>>
>>>
>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>
>>
>>
>>
>> --
>> 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
>
--
*Akshay Joshi*
*Sr. Software Architect *
*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-05-22 09:13:47 | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 |
Previous Message | Aditya Toshniwal | 2018-05-22 08:46:21 | Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database. |