Re: [PHP-DEV] Expose pdo_parse_params to userspace

2023-10-18 Thread Matteo Beccati
Hi Karoly, Il 11/10/2023 17:03, Karoly Negyesi ha scritto: Drupal has been using PDO for a long time now but as it doesn't have async support, the project is adding a mysqli driver. A sticky point here is the lack of support for named placeholders which are widely used in Drupal land but mysqli

Re: [PHP-DEV] Expose pdo_parse_params to userspace

2023-10-12 Thread Robert Landers
On Wed, Oct 11, 2023 at 8:36 PM Karoly Negyesi wrote: > > That's rather concerning on both fronts. > > Drupal doesn't plan on abandoning PDO, I believe. At least I haven't seen > that proposed. > > Is there anything I could help with fixing pdo_parse_params ? > > Also what's the problem with

RE: [PHP-DEV] Expose pdo_parse_params to userspace

2023-10-11 Thread Jeffrey Dafoe
> I have to say I am not a fan of this proposal. While definitely a super nice > feature in PDO, it's more of a hack rather than proper feature. > Certain RDBMSs support named parameters in prepared statements, but > MySQL doesn't. Therefore, the solution implemented in PDO is a hack. It's very

Re: [PHP-DEV] Expose pdo_parse_params to userspace

2023-10-11 Thread Karoly Negyesi
That's rather concerning on both fronts. Drupal doesn't plan on abandoning PDO, I believe. At least I haven't seen that proposed. Is there anything I could help with fixing pdo_parse_params ? Also what's the problem with mysqli async? With fibers introduced in 8.1 it seems a very good fit ,

Re: [PHP-DEV] Expose pdo_parse_params to userspace

2023-10-11 Thread Kamil Tekiela
Hi, I have to say I am not a fan of this proposal. While definitely a super nice feature in PDO, it's more of a hack rather than proper feature. Certain RDBMSs support named parameters in prepared statements, but MySQL doesn't. Therefore, the solution implemented in PDO is a hack. It's very

[PHP-DEV] Expose pdo_parse_params to userspace

2023-10-11 Thread Karoly Negyesi
Hello, Drupal has been using PDO for a long time now but as it doesn't have async support, the project is adding a mysqli driver. A sticky point here is the lack of support for named placeholders which are widely used in Drupal land but mysqli doesn't support them. Currently the project is