From: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Sahil Harpal <sahilharpal1234(at)gmail(dot)com> |
Cc: | pgadmin-hackers(at)postgresql(dot)org, Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, Dave Page <dpage(at)pgadmin(dot)org>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Subject: | Re: Pgadmin4 System Stats Extension Design |
Date: | 2023-07-27 10:56:46 |
Message-ID: | CAFOhELdT30L7OYj3MbQtkaHLHaRuMP3inV4t_wHkiJBOiJ=8TA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Thu, Jul 27, 2023 at 1:59 PM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi Sahil,
>
> On Thu, Jul 27, 2023 at 1:09 PM Sahil Harpal <sahilharpal1234(at)gmail(dot)com>
> wrote:
>
>> On Wed, 12 Jul 2023 at 15:58, Sahil Harpal <sahilharpal1234(at)gmail(dot)com>
>> wrote:
>>
>>> On Wed, 12 Jul 2023 at 10:47, Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>> [image: image.gif]
>>>
>>>> It fails while applying. Can you please rebase your patch and send it?
>>>>
>>>
>>> Could you please try the attached patch.
>>>
>> Just a follow-up email to check if I can proceed with a similar structure
>> and logic for the remaining disk tab.
>>
> You have made different files tab-wise; it would be more manageable if you
could put them under a subfolder OR start a name with system_state or
something like that.
> If anyone has faced any issues while using it or has any suggestions for
>> improvement, please let me know, and I will make the necessary changes and
>> follow the same approach for the disk tab. Although there is an issue with
>> drive's letters, the I/O analysis of block devices is working just fine on
>> my system, so I can probably start working on it parallelly.
>>
>> Some initial review comments.
>
> - While applying the patch, I got warnings; please fix those.
> - Fix PEP-8 errors (yarn run pep8)
> - Fix the linter errors. Run *yarn run bundle:dev* instead of *yarn run
> webpacker so* you will get the linter errors on every run.
> - If the System Stat extension does not exist, display the appropriate
> message instead of blank graphs.
> - For the first time, graphs start from the opposite side, and after the
> following API call, those get adjusted; please fix that.
> - Please consider the refresh rate for the API calls (preferences settings
> > Dashboards > Refresh rates)
>
> The code review is in progress; I will send my review comments once
> completed.
>
> Thanks,
> Khushboo
>
>
>> Thanks,
>> Sahil
>>
>>
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2023-07-27 12:04:38 | [pgadmin-org/pgadmin4] a46064: 1) Added support for SYSTEM, CONCURRENTLY and TABL... |
Previous Message | dependabot[bot] | 2023-07-27 10:54:21 | [pgadmin-org/pgadmin4] d663f6: Python dependency: Update flask-security-too requi... |