From: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Josh Berkus <josh(at)agliodbs(dot)com>, pgAdmin Support <pgadmin-support(at)postgresql(dot)org> |
Subject: | Re: Please add timestamps to pgagent log |
Date: | 2015-07-16 07:19:00 |
Message-ID: | CA+yw=mOhgbJr3QN10MKfmBx4nNqtZzT0G9gQAsP+UyqbkGvwdw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
Hi Dave,
I have created a new patch with changes you have suggested.
Please find it attached with this mail.
Please do review it.
Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb
On Mon, Jul 13, 2015 at 7:06 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> HI
>
> On Mon, Jul 13, 2015 at 1:59 PM, Sanket Mehta
> <sanket(dot)mehta(at)enterprisedb(dot)com> wrote:
> > Hi ,
> >
> > I have attached the patch which is having both functionalities (-v for
> > version info and timestamp in log)
> >
> > Here timestamp is added only for linux platform because on windows
> pgagent
> > logs are viewed in event viewer which provides timestamp on its own.
>
> OK
>
> > Please do review it and let me know in case anything is missing.
>
> Can we not get the version from a preprocessor macro? It's already in
> CMakeLists.txt after all. If not, I think we at least need to update
> the comment in CMakeLists.txt and other files as appropriate so we
> don't forget to update the number anywhere.
>
> Also, please change:
>
> shows version then exit
>
> to
>
> display version info and then exit.
>
> Thanks!
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
Attachment | Content-Type | Size |
---|---|---|
pgagent_version_timestampv2.patch | application/octet-stream | 4.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Michiel Hendriks | 2015-07-16 07:27:37 | Disable password caching for two factor authentication |
Previous Message | Sanket Mehta | 2015-07-14 05:37:29 | Re: pgagent hangs forever in "r" state |