From: | Zaid Shabbir <zaidshabbir(at)gmail(dot)com> |
---|---|
To: | Pravesh Sharma <pravesh(dot)sharma(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Regarding issue #6510 |
Date: | 2025-05-20 15:49:29 |
Message-ID: | CABCJe_UOGCZKQm5pY2rv8_WonP9ga-b2B3MXbBbDDOUv5pBFUg@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hello Pravesh
Nice implementation. In case someone wants to review the complete JSON, is
there a way to open the contents in a separate window (popup or side
window) on a click event? The entire larger string can be loaded on demand
when the click event is triggered.
Regards,
Zaid
On Tue, May 20, 2025 at 10:16 AM Pravesh Sharma <
pravesh(dot)sharma(at)enterprisedb(dot)com> wrote:
> Hi Hackers,
>
> I am working on issue #6510
> <https://github.com/pgadmin-org/pgadmin4/issues/6510>, where the result
> grid slows down and sometimes hangs when large JSON or text data is present
> in a table column. Upon investigation, I found that this is due to
> rendering the entire JSON to the DOM. This causes slowness, especially with
> large datasets; for example, rendering a 15MB JSON object for preview
> purposes is unnecessary. I am proposing a solution where the result grid
> will render data up to a certain length. I have implemented these changes,
> and everything is working fine, resulting in a significant performance
> improvement. As you can see in the screenshot, the JSON data is 20MB, and
> on the result grid, we are displaying only 200 characters as it is only a
> preview. I am also considering making this maximum display length a user
> preference, named "Max column data display length," allowing them to
> customize how much data they want to see. Can you tell me what you think of
> this approach or if you have any suggestions?
>
>
> Thanks,
>
> Pravesh
>
> [image: image.png]
> --
>
>
> Pravesh Sharma
>
> Senior SDE
>
> +91 9406461406
>
>
> enterprisedb.com
>
From | Date | Subject | |
---|---|---|---|
Next Message | Pravesh Sharma | 2025-05-21 04:34:20 | Re: Regarding issue #6510 |
Previous Message | Usman Khan | 2025-05-20 15:37:46 | Re: Regarding issue #6510 |