Re: [pgAdmin4][RM#2950] Add marker to dashboard queries

Поиск
Список
Период
Сортировка
От Murtuza Zabuawala
Тема Re: [pgAdmin4][RM#2950] Add marker to dashboard queries
Дата
Msg-id CAKKotZScCr0BE_PrgwfHeYLyQ4fPDsG2Ms6m2XgoabkfWqLdPA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][RM#2950] Add marker to dashboard queries  (Joao De Almeida Pereira <jdealmeidapereira@pivotal.io>)
Ответы Re: [pgAdmin4][RM#2950] Add marker to dashboard queries
Список pgadmin-hackers
Hi Joao,

It was mainly intended for PostgreSQL database logs and not pgAdmin4 logs :)


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Thu, Feb 15, 2018 at 8:03 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Murtuza,
I was looking into the Issue information and the solution that was provided there needs some polish. With the solution that was asked, the log will still be generated, but eventually will not show up in a log monitor, but nevertheless will be there.

After some digging around the code I saw that we are using Logging Level 25 for all SQL queries what if we give 24 for the dashboard logs?
Another option, that I didn't look into very much was the possibility of passing a logger into the execution functions or even somehow tag them so we can have more control over the logs that we generate. It would be nice if we could somehow ignore all the logs from the Dashboard. (http://flask.pocoo.org/docs/dev/logging/#other-libraries)

What do you think about the suggestion?

Thanks
Joao

On Thu, Feb 15, 2018 at 8:51 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to add marker in dashboard queries, it will allow to user to filter out those queries from database logs if required.

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


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

Предыдущее
От: Joao De Almeida Pereira
Дата:
Сообщение: Re: [pgAdmin4][RM#3121] Fix PEP-8 issues
Следующее
От: Murtuza Zabuawala
Дата:
Сообщение: Re: [pgAdmin4][RM#3121] Fix PEP-8 issues