Re: [pgAdmin4][Patch]: Fixed RM #2489: Copy from the results grid is extremely slow and fails with large datasets

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: Fixed RM #2489: Copy from the results grid is extremely slow and fails with large datasets
Date: 2017-06-29 13:13:33
Message-ID: CAKKotZRs=MW93vE9zP6HtZrjHoHt_=aC-pDaCcQ7LbETqkR=3w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Yes, that's weird issue, copy command works fine with debugger on and fails
without debugger.
I used below code to check the status of copy command.

try {
var status = document.execCommand('copy');
console.log('Copying text command was ' + (status ?
'successful' : 'unsuccessful'));
} catch (err) {
alert('Oops, unable to copy to clipboard');
}

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Thu, Jun 29, 2017 at 6:10 PM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

> Hi,
>
> I haven't come across this error while development as I tested the patch
> with Javascript debugger on.
> When I put a single break-point and then execute the code, it's giving the
> perfect output but not without a debugger.
>
> I have been trying to find the solution since yesterday but couldn't. I
> have tried many things but didn't work for me.
> So, if anyone has any clue please let me know as I am still trying.
>
> Thanks,
> Khushboo
>
>
>
>
>
>
>
> On Wed, Jun 28, 2017 at 7:25 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Wed, Jun 28, 2017 at 9:48 AM, Harshal Dhumal <
>> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Khushboo,
>>>
>>> I applied your patch and tried to copy result from grid but got below
>>> exception.
>>> [image: Inline image 1]
>>> Also when I pressed copy button to copy 100k rows the browser was
>>> unresponsive for ~18 seconds (though copy operation was failed).
>>>
>>
>> Yes, I see the same error, no noticeable speedup, and nothing left on the
>> clipboard.
>>
>>
>>>
>>> --
>>> *Harshal Dhumal*
>>> *Sr. Software Engineer*
>>>
>>> EnterpriseDB India: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Wed, Jun 28, 2017 at 6:20 PM, Khushboo Vashi <
>>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> Please find the attached patch to fix the RM #2489: Copy from the
>>>> results grid is extremely slow and fails with large datasets.
>>>>
>>>> Thanks,
>>>> Khushboo
>>>>
>>>
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Joao De Almeida Pereira 2017-06-29 13:25:14 Re: [pgadmin-hackers][Patch] Change alert styling
Previous Message Khushboo Vashi 2017-06-29 12:40:26 Re: [pgAdmin4][Patch]: Fixed RM #2489: Copy from the results grid is extremely slow and fails with large datasets