Re: [PHP-DEV] PDO fetch performance problems with many bind parameters

2020-08-28 Thread Matteo Beccati
Hi George, /cc release managers - pls don't hate me ;-) On 27/08/2020 10:23, Matteo Beccati wrote: > > On 26/08/2020 19:28, G. P. B. wrote: >> 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 rig

Re: [PHP-DEV] PDO fetch performance problems with many bindparameters

2020-08-28 Thread Christoph M. Becker
On 28.08.2020 at 09:18, Matteo Beccati wrote: > Hi George, > > /cc release managers - pls don't hate me ;-) > > On 27/08/2020 10:23, Matteo Beccati wrote: >> >> On 26/08/2020 19:28, G. P. B. wrote: >>> This can still target PHP 8.0 as this seems to be akin to a bug this >>> might even be applicabl

Re: [PHP-DEV] PDO fetch performance problems with many bind parameters

2020-08-28 Thread Gabriel Caruso
On Fri, 28 Aug 2020 at 09:19, Matteo Beccati wrote: > Hi George, > > /cc release managers - pls don't hate me ;-) > > On 27/08/2020 10:23, Matteo Beccati wrote: > > > > On 26/08/2020 19:28, G. P. B. wrote: > >> This can still target PHP 8.0 as this seems to be akin to a bug this > >> might even b

Re: [PHP-DEV] PDO fetch performance problems with many bind parameters

2020-08-28 Thread Dino Pejaković
The script I used to pinpoint this problem and test my temporary fix, and later Matteo's PR is linked in the bug report I oppened. Should be simple to run if anyone is interested in trying. I found this on pdo_pgsql, but looking at the code it looks like the problem should happen on other pdo_*

Re: [PHP-DEV] PDO fetch performance problems with many bindparameters

2020-08-28 Thread Christoph M. Becker
On 28.08.2020 at 20:36, Matteo Beccati wrote: > Hi Gabriel, > >> Let's have this patch merged into PHP 8.0 (`master` branch as of today), I >> have nothing against it.> >> About PHPs 7.3 and 7.4: if the performance improvements are this massive as >> describe in this email, I'm :+1: to have it on

Re: [PHP-DEV] PDO fetch performance problems with many bind parameters

2020-08-28 Thread Matteo Beccati
Hi Gabriel, > Let's have this patch merged into PHP 8.0 (`master` branch as of today), I > have nothing against it.> > About PHPs 7.3 and 7.4: if the performance improvements are this massive as > describe in this email, I'm :+1: to have it on those versions as well. The mileage may very: running