Re: Regarding issue #6510

From: Zaid Shabbir <zaidshabbir(at)gmail(dot)com>
To: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
Cc: Pravesh Sharma <pravesh(dot)sharma(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Regarding issue #6510
Date: 2025-05-21 06:46:56
Message-ID: CABCJe_WB0twujfuC0fLfPY+Q2RGS1Sz6R26Ld79mgwHtxY30zQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

Nice to hear about the implementation. Overall, the grid looks fine to me,
with one minor adjustment related to the value JSONB default column width:
I suggest reducing the *default width* to approximately half its current
size, since—if someone wants to view the full JSON—they can simply
double-click to expand it. In this scenario, a very wide column looks bulky
on the screen.

Regards,
Zaid

On Wed, May 21, 2025 at 10:27 AM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> Hi,
>
> To add what Pravesh mentioned - There is no change in any existing
> behaviour. The user can still copy and edit everything as before. The only
> change is what is visible on the result grid, which will also be
> customisable. Most users will anyway prefer to double click and open to see
> the data instead of scrolling horizontally a large column.
>
> On Wed, May 21, 2025 at 10:05 AM Pravesh Sharma <
> pravesh(dot)sharma(at)enterprisedb(dot)com> wrote:
>
>> Hi Zaid,
>>
>> We do already have that, this implementation is only for data grid when
>> we click on the cell we can see the entire JSON content.
>>
>> Thanks,
>> Pravesh
>>
>> On Tue, May 20, 2025 at 9:19 PM Zaid Shabbir <zaidshabbir(at)gmail(dot)com>
>> wrote:
>>
>>> 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
>>>>
>>>
>
> --
> Thanks,
> Aditya Toshniwal
> pgAdmin Hacker | Sr. Staff SDE II | *enterprisedb.com*
> <https://www.enterprisedb.com/>
> "Don't Complain about Heat, Plant a TREE"
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Aditya Toshniwal 2025-05-22 07:02:02 Open preferences in new tab
Previous Message Aditya Toshniwal 2025-05-21 05:26:24 Re: Regarding issue #6510