From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Gilles Darold <gilles(dot)darold(at)dalibo(dot)com> |
Cc: | Daniel Verite <daniel(at)manitou-mail(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: proposal: psql \setfileref |
Date: | 2016-10-11 07:40:14 |
Message-ID: | CAFj8pRCkiGatDoAnM-KJuJaUXzCs3P2GNvh8OrTqp8LY-8exDw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2016-10-11 9:32 GMT+02:00 Gilles Darold <gilles(dot)darold(at)dalibo(dot)com>:
> Le 11/10/2016 à 07:53, Pavel Stehule a écrit :
>
>
>
> 2016-10-10 19:50 GMT+02:00 Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>:
>
>>
>>
>> 2016-10-10 15:17 GMT+02:00 Gilles Darold <gilles(dot)darold(at)dalibo(dot)com>:
>>
>>> Le 10/10/2016 à 14:38, Daniel Verite a écrit :
>>> > Gilles Darold wrote:
>>> >
>>> >> postgres=# \setfileref b /dev/random
>>> >> postgres=# insert into test (:b);
>>> >>
>>> >> Process need to be killed using SIGTERM.
>>> > This can be fixed by setting sigint_interrupt_enabled to true
>>> > before operating on the file. Then ctrl-C would be able to cancel
>>> > the command.
>>>
>>> If we do not prevent the user to be able to load special files that
>>> would be useful yes.
>>>
>>
>> I don't think so special files has some sense, just I had not time fix
>> this issue. I will do it early - I hope.
>>
>
> fresh patch - only regular files are allowed
>
> Regards
>
> Pavel
>
>
> Hi,
>
> The patch works as expected, the last two remaining issues have been
> fixed. I've changed the status to "Ready for committers".
>
> Thanks Pavel.
>
Thank you very much
Regards
Pavel
> --
> Gilles Darold
> Consultant PostgreSQLhttp://dalibo.com - http://dalibo.org
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2016-10-11 08:00:07 | Re: proposal: schema PL session variables |
Previous Message | Gilles Darold | 2016-10-11 07:32:52 | Re: proposal: psql \setfileref |