Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:100805
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain bohwaz.net designates 5.135.163.151 as permitted sender)
Date: Sun, 1 Oct 2017 18:55:40 +1300
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: internals@lists.php.net
Message-ID: <20171001185540.5e572e58@platypus>
In-Reply-To: <0607d622-caa9-ae1d-33db-1958a0ff0500@gmx.de>
References: <25903b1e0fc5a726e44b1e3a5ab4aec4@bohwaz.net>
	<0607d622-caa9-ae1d-33db-1958a0ff0500@gmx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Subject: Re: [RFC] [Discussion] Implement SQLite "openBlob" feature in PDO
From: php@bohwaz.net (BohwaZ)

On Wed, 27 Sep 2017 11:41:50 +0200 / "Christoph M. Becker"
<cmbecker69@gmx.de> said :

> On 26.09.2017 at 04:03, BohwaZ/PHP wrote:
> 
> > following my patch and discussions on this list, here is the RFC
> > requested by some people here to implement "openBlob" in the
> > pdo_sqlite driver, to match the "openBlob" method from the SQLite3
> > extension.
> > 
> > https://wiki.php.net/rfc/implement_sqlite_openblob_in_pdo
> > 
> > Discussion should happen in the next two weeks before going to vote.
> > 
> > The actual patch is here: https://github.com/php/php-src/pull/2698
> > 
> > Suggestions and discussions welcome.
> 
> PDO already has support for large objects (LOBs)[1].  I don't know if
> and how these are supported by the pdo_sqlite driver, but wouldn't it
> make sense to use the existing API instead of introducing a new
> method?
> 
> [1] <http://www.php.net/manual/en/pdo.lobs.php>
> 

Very interesting indeed, didn't know about that feature, I was
expecting the creation of a new method was the only way, as this was
the way PGSQL was doing it.

There's even a bug report about it:
https://bugs.php.net/bug.php?id=57691

I will look into that next week and see if it can fit and replace my RFC
then.

Thank you.