From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com> |
Subject: | Re: [pgAdmin4] RM5965 Couldn't download file of Marcos query results |
Date: | 2020-12-09 08:23:00 |
Message-ID: | CANxoLDc+3YGFSJ8hvuJjVBVBTCe=yFDov_DqTGa5D30m0wKeAQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Aditya
Can you please review this patch?
On Mon, Dec 7, 2020 at 3:10 PM Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
wrote:
> Hi Hackers,
>
> Please find the updated patch below.
>
> On Mon, Dec 7, 2020 at 2:55 PM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Please find the attached patch which resolves the issue of macros query
>> results download, have used async cursor to achieve this functionality,
>> where for downloading the results, cursor is scrolled back to 0 and end of
>> the records, and reset again while user scrolling on Data Output table.
>>
>> QA/Reviewer needs to observe below issues if it occurs:
>>
>> 1. If records are more like 5000 or 10000, try to fetch records by
>> scrolling at least (2000), now save the results by clicking the download
>> button, it should save the file, and now try scrolling again, the data
>> should be shown continuously and not any abrupt end or unexpected records
>> order.
>> 2. Also, the download button is now "Save results to CSV/TXT" where
>> it will only get enabled when there are valid records in the Data Output.
>>
>>
>> Also a minor fix of the add folder icon issue is also added into this
>> patch.
>>
>> --
>> *Rahul Shirsat*
>> Senior Software Engineer | EnterpriseDB Corporation.
>>
>
>
> --
> *Rahul Shirsat*
> Software Engineer | EnterpriseDB Corporation.
>
--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres <http://edbpostgres.com>*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Nikhil Mohite | 2020-12-09 09:04:57 | [pgAdmin][RM-6058]: Renaming feature should be disabled for files opened in tab. |
Previous Message | Akshay Joshi | 2020-12-09 08:17:59 | Re: pgAdmin4 v4.29 candidate builds |