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"
>
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 |