Re: PATCH: Fix the issue for saving query output as CSV - Mailing list pgadmin-hackers
Hi,
Please find attached patch file to support server side download in runtime application.
NOTE: Before applying this patch, use above harshal's patch.
Thanks,
Neel Patel
On Mon, Jul 18, 2016 at 5:31 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,PFA rebased patch RM1405 (version 4)Neel will be sending patch for QT issue for same functionality.--Harshal DhumalSoftware EngineerOn Mon, Jul 18, 2016 at 4:02 PM, Dave Page <dpage@pgadmin.org> wrote:Perfect, thanks!On Mon, Jul 18, 2016 at 11:24 AM, Neel Patel <neel.patel@enterprisedb.com> wrote:Yes. I am just testing with different OS. I will send it next 1-2 hour.Is it fine ?Thanks,Neel PatelOn Mon, Jul 18, 2016 at 3:53 PM, Dave Page <dpage@pgadmin.org> wrote:Any chance of getting that in the next couple of hours so I can get it into beta 3?On Mon, Jul 18, 2016 at 11:18 AM, Dave Page <dpage@pgadmin.org> wrote:Thanks!On Mon, Jul 18, 2016 at 10:42 AM, Neel Patel <neel.patel@enterprisedb.com> wrote:Hi Dave,Yes, it will break in runtime. If we need to support downloading at server side then we need to change the runtime code.I will send the patch for runtime to support server side download CSV file.Thanks,Neel PatelOn Fri, Jul 15, 2016 at 3:05 PM, Dave Page <dpage@pgadmin.org> wrote:HiThis seems to break downloads in the runtime. It works in Safari and Chrome, but in the runtime the suggested filename is "6980287?query=SELECT+*+FROM+pem.probe_column%0AORDER+BY+id%0AASC+&filename=probe_column.csv", and after changing that and hitting OK, the "Downloading file" dialogue is shown indefinitely.On Wed, Jul 13, 2016 at 1:16 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Hi,PFA rebased patch for RM1405--Harshal DhumalSoftware EngineerOn Wed, Jul 13, 2016 at 5:15 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:Yes sure--Harshal DhumalSoftware EngineerOn Wed, Jul 13, 2016 at 5:10 PM, Dave Page <dpage@pgadmin.org> wrote:HiCan you rebase this please?Thanks.On Mon, Jul 11, 2016 at 9:16 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Changes: To download query result to CSV directly from server and not to use download attr. of anchor tag (<a>) as it's not support by all of major browsers (e.g. Safari).
Also it's not feasible to load data in html to download if result set of query is very huge (in GBs).--Harshal DhumalSoftware EngineerOn Wed, Jun 29, 2016 at 4:53 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:On Wed, Jun 29, 2016 at 3:52 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:Yes, It will not work in runtime as well but I think Neel is working for fix in run time for this issue, We might able to fix it in run time but issue persists in Safari unless they add support in browser itself.
https://webkit.org/status/#feature-download-attribute
> On 29-Jun-2016, at 3:40 pm, Dave Page <dpage@pgadmin.org> wrote:
>
> On Tue, Jun 28, 2016 at 10:33 AM, Murtuza Zabuawala
> <murtuza.zabuawala@enterprisedb.com> wrote:
>> Yes Dave, I agree that downloading files has been supported in browsers since long .
>>
>> But in general we send request & then receives files from web server but in our case we are fetching our data from Backbone models & then converting it to CSV format for downloading as a file at client side in browser itself.
>
> If Safari doesn't support client-side saving of files, then I have to
> wonder if our runtime will either - both are webkit based.
>
> So I guess the next question to ask is; why don't we just generate the
> CSV on the server side?
>
@Akshay,
Can you please suggest on above?As we already have complete data in backgrid's full collection, so I have used it instead of fetching it again.
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers----Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment
pgadmin-hackers by date: