Re: [pgAdmin4][RM#3072] Make pgagent job history rows configurable

From: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM#3072] Make pgagent job history rows configurable
Date: 2018-04-05 11:22:30
Message-ID: CAFOhELcV+0nNx8_7RxMR-FuhmpudjQXL=KowT-rYiUo4jvzaUw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Thu, Apr 5, 2018 at 4:43 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Thu, Apr 5, 2018 at 11:10 AM, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> Hi,
>>
>> PFA patch which allow user to configure how many rows they wish to
>> display for any pgagent jobs on statistics panel.
>>
>
> I think this is essentially good, however, I'm really not happy with the
> preference name and category. In general, I'd suggest that before creating
> patches in the future we should confirm naming etc on the mailing list, as
> I often end up changing wording and then requiring new screenshots etc.
>
> +1

> In this case, I really don't like that we've added another category, and
> quite a specific one at that. I would suggest we move it to Browser ->
> Properties and name it "Maximum job history rows" with a description of
> "The maximum number of history rows to show on the Statistics tab for
> pgAgent jobs."
>
> Thoughts?
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2018-04-05 11:22:39 Re: [pgAdmin4][RM#3072] Make pgagent job history rows configurable
Previous Message Dave Page 2018-04-05 11:13:25 Re: [pgAdmin4][RM#3072] Make pgagent job history rows configurable