Re: [pgAdmin4][RM#2950] Add marker to dashboard queries - Mailing list pgadmin-hackers

From Joao De Almeida Pereira
Subject Re: [pgAdmin4][RM#2950] Add marker to dashboard queries
Date
Msg-id CAE+jjanFb+Np3BqmPDzB80cHJzM4uEZNH2bge+0LS7Q+j6EczA@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][RM#2950] Add marker to dashboard queries  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Responses Re: [pgAdmin4][RM#2950] Add marker to dashboard queries
List pgadmin-hackers
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 by date:

Previous
From: Robert Eckhardt
Date:
Subject: Next release
Next
From: Joao De Almeida Pereira
Date:
Subject: Re: [pgAdmin4][RM#3121] Fix PEP-8 issues