Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:111699
To: "G. P. B." <george.banyard@gmail.com>
Cc: =?UTF-8?Q?Dino_Pejakovi=c4=87?= <dino.pejakovic@voxdiversa.hr>,
 PHP internals <internals@lists.php.net>
References: <08b959f5-7652-5e8a-562a-c2659facec8c@voxdiversa.hr>
 <c2e43e61-bc14-2b0f-6fdd-28235153ced8@beccati.com>
 <CAFPFaMJTErRzCuA5RWAsfPmqUr9Hx22+4RuCH9NKKioVgf+3OA@mail.gmail.com>
Message-ID: <20277058-b463-038a-4333-eacd023ccdc4@beccati.com>
Date: Thu, 27 Aug 2020 10:23:02 +0200
MIME-Version: 1.0
In-Reply-To: <CAFPFaMJTErRzCuA5RWAsfPmqUr9Hx22+4RuCH9NKKioVgf+3OA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-GB
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] PDO fetch performance problems with many bind
 parameters
From: php@beccati.com (Matteo Beccati)

Hi George,

On 26/08/2020 19:28, G. P. B. wrote:
> Just that you are aware the emails from both of you went to my spam folder.

Thanks. Alas, big G doesn't like the fact I'm using my own email server,
despite all DKIM/SPF/whatnot being properly configured, according to them.

> Back to the matter, this is the correct list to email Dino, however could
> you
> open a pull request on GitHub for this change so that more eyes (and
> especially eyes from people which know PDO) can have a look?
> 
> Please provide the relevant information again in the pull request and
> possibly host your files on a gist.
> 
> This can still target PHP 8.0 as this seems to be akin to a bug this
> might even be applicable to PHP 7.3/7.4. Anyways, good catch.

You could be right. I've created a draft PR for review: Dino could you
please benchmark it and get back with results?

https://github.com/php/php-src/pull/6047


Cheers
-- 
Matteo Beccati

Development & Consulting - http://www.beccati.com/