Re: [pgAdmin4] RM5965 Couldn't download file of Marcos query results

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

In response to

Responses

Browse pgadmin-hackers by date

  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