Re: Pgadmin4 System Stats Extension Design

From: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
To: Sahil Harpal <sahilharpal1234(at)gmail(dot)com>
Cc: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers(at)postgresql(dot)org, 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-08-16 04:18:21
Message-ID: CAFOhELf_c5Uz3Q2+oqGqPsnOgE9S4SSchdG2MdOoe4yyWTMpZg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Can you please rebase the patch and send it again?

On Thu, Aug 10, 2023 at 12:37 AM Sahil Harpal <sahilharpal1234(at)gmail(dot)com>
wrote:

> Hi all,
>
> I have attached the updated patch with recent changes.
>
> New changes:
>
> 1. Displayed the appropriate message If the System Stat extension does
> not exist.
> 2. Provided option to configure refresh rates for the API calls
> (Preferences settings > Dashboards > Refresh rates).
> 3. Added I/O analysis of block devices under the storage tab.
> 4. Fixed PEP-8 errors.
> 5. Fixed the linter errors.
> 6. Moved all the system stats related .jsx files under a
> seperate folder "dashboard/static/js/SystemStats".
> 7. Added formatter to convert disk sizes (in bytes) to human readable
> format.
>
> Pending Work:
>
> 1. Process information -
> - Issue: The pg_sys_process_info() query takes much longer (more
> than 2 mins) to execute and prevents the updation of other graphs and
> tables.
> 2. Disk information -
> - Issue: The pg_sys_disk_info() query returns NULL value for some
> of the drive letters.
> 3. StreamingChart -
> - Issue: Graph shifting glitch. For the first time, graphs start
> from the opposite side, and after the following API call or a few seconds
> later, those get adjusted.
>
> I need suggestions for labels for different tables and charts. Also, could
> you please clarify the use of the counterData variable, which is used for
> some of the charts (tps_stats, ti_stats, to_stats, and bio_stats)?
>
> Thank you,
> Sahil
>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Sahil Harpal 2023-08-16 05:38:49 Re: Pgadmin4 System Stats Extension Design
Previous Message Sahil Harpal 2023-08-16 02:59:58 Re: Pgadmin4 System Stats Extension Design