Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database.

Поиск
Список
Период
Сортировка
От Aditya Toshniwal
Тема Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database.
Дата
Msg-id CAM9w-_=c=82+5ESAEMXUQf5NOrfouZ5JxJMfr+56vJAfGz2DrA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database.  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Ответы Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database.  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Hi Hackers,

PFA updated patch. The sqleditor change is sent separately and removed from current patch as suggested.
The test cases were running fine when the module was specified using --pkg but were failing in complete run. Fixed that.

Kindly review.

Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

On Tue, Jun 5, 2018 at 10:15 AM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi

On Tue, Jun 5, 2018 at 1:08 AM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.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


Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgadmin4][Patch]: Test cases for the backup module
Следующее
От: Khushboo Vashi
Дата:
Сообщение: Re: [pgadmin4][Patch]: Test cases for the backup module