From: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
---|---|
To: | Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database. |
Date: | 2018-06-05 04:45:33 |
Message-ID: | CAM9w-_km5Zv=Gw6CUZF8m3zzacOKL-ir-QNw3xPjP2MJfLRg7w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Tue, Jun 5, 2018 at 1:08 AM, Joao De Almeida Pereira <
jdealmeidapereira(at)pivotal(dot)io> wrote:
> Hello Aditya,
>
>>
>>
>> There is no change related to notifications in this patch.
>> The below code is minor fix related to connection status of sql editor.
>> Can you please share the code snippet if it is not the below.
>>
>> - # Check for the asynchronous notifies statements.
>> - conn.check_notifies(True)
>> - notifies = conn.get_notifies()
>> + if status is not None:
>> + # Check for the asynchronous notifies statements.
>> + conn.check_notifies(True)
>> + notifies = conn.get_notifies()
>>
>>
> This is a minor fix, but is it related to querying SQL_ASCII database?
>
No its not. It is something I found when I was working on SQL_ASCII related
changes.
Well then, will send a separate patch for it.
>
> Thanks
> Victoria && Joao
>
From | Date | Subject | |
---|---|---|---|
Next Message | Aditya Toshniwal | 2018-06-05 04:49:21 | [pgAdmin4][patch] sqleditor connection status minor patch |
Previous Message | Ashesh Vashi | 2018-06-05 04:07:00 | Re: pgAgent 4.0 patch |