From: | Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com> |
---|---|
To: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4] RM5965 Couldn't download file of Marcos query results |
Date: | 2020-12-07 09:39:53 |
Message-ID: | CAKtn9dPWeCXuToAyOSE3Mqdg-b0Qkx3vcRX30nuUrxabY-O-+Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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.
Attachment | Content-Type | Size |
---|---|---|
RM5965_v2.patch | application/octet-stream | 20.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Nikhil Mohite | 2020-12-07 10:02:02 | Re: [pgAdmin][RM-5343]: [Code Coverage] Improve API test cases for Debugger |
Previous Message | Rahul Shirsat | 2020-12-07 09:34:24 | Add folder icon issue |