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

Hello Murtuza,
Looks like I missread the Issue then.
In that case I think it might be the solution that we can implement. Unless
anyone in the list as a different idea.

Thanks
Joao

On Thu, Feb 15, 2018 at 11:12 AM Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> 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(at)pivotal(dot)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(dot)zabuawala(at)enterprisedb(dot)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
>>>
>>>
>