From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Noah Misch <noah(at)leadboat(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: lo_create(oid, bytea) breaks every extant release of libpq |
Date: | 2014-06-12 05:19:30 |
Message-ID: | CAFj8pRCxvfRv1DTJJc0EXB3ke9BXiu_7pY_o6L1RvCp4fReTtw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2014-06-12 7:08 GMT+02:00 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
> Noah Misch <noah(at)leadboat(dot)com> writes:
> > On Wed, Jun 11, 2014 at 11:57:20PM -0400, Tom Lane wrote:
> >> Meanwhile, we have to either revert the addition of lo_create(oid,
> >> bytea) altogether, or choose a different name for it. Suggestions?
>
> > lo_new() or lo_make()? An earlier draft of the patch that added
> > lo_create(oid, bytea) had a similar function named make_lo().
>
> I think we want to stick to the lo_xxx naming convention, whatever
> xxx ends up being.
>
> I was idly thinking that we might want to focus on the fact that this
> function not only creates a LO but loads some data into it. lo_make
> isn't too bad, but we could also consider lo_load, lo_import, etc.
> (lo_import is not one of the names we have to avoid overloading.
> OTOH, there's already a 2-argument form of it, so maybe there'd be
> issues with resolving calls with unknown-literal arguments.)
>
>
I have not any problem with lo_new, lo_make. lo_import is related to import
from host system. I am not sure about lo_load, but I am not able to specify
arguments why not.
Pavel
> regards, tom lane
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2014-06-12 06:38:12 | Re: Something flaky in the "relfilenode mapping" infrastructure |
Previous Message | Tom Lane | 2014-06-12 05:18:22 | Re: Something flaky in the "relfilenode mapping" infrastructure |