Re: lo_create(oid, bytea) breaks every extant release of libpq

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
>

In response to

Browse pgsql-hackers by date

  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