- Issue created by @mondrake
- 🇮🇹Italy mondrake 🇮🇹
Patch built on top of 🐛 Implement statement classes using \Iterator to fix memory usage regression and prevent rewinding Fixed , suggest to wait for that to happen first.
- @mondrake opened merge request.
- Status changed to Needs review
over 1 year ago 8:45pm 17 March 2023 - 🇮🇹Italy mondrake 🇮🇹
Parent got committed. Put the patch in a MR, for review.
- Status changed to RTBC
over 1 year ago 10:17pm 17 March 2023 - 🇳🇱Netherlands daffie
All code changes look good to me.
We already have testing for all the changes in the database FetchTest.
For me it is RTBC. - 🇮🇹Italy mondrake 🇮🇹
FWIW, just implemented the trait in the mysqli driver on Github, https://github.com/mondrake/mysqli/commit/52da30ef0f1ab47d18ace0c7f4e89d...
All good.
- last update
over 1 year ago 29,202 pass - last update
over 1 year ago 29,207 pass - last update
over 1 year ago 29,283 pass - last update
over 1 year ago 29,302 pass - last update
over 1 year ago 29,302 pass - last update
over 1 year ago 29,300 pass 12:01 8:31 Running- last update
over 1 year ago 29,366 pass - last update
over 1 year ago 29,367 pass - last update
over 1 year ago 29,374 pass - last update
over 1 year ago 29,379 pass - last update
over 1 year ago 29,380 pass - last update
over 1 year ago 29,380 pass - last update
over 1 year ago 29,383 pass - last update
over 1 year ago 29,388 pass - last update
over 1 year ago 29,388 pass - last update
over 1 year ago 29,388 pass - last update
over 1 year ago 29,388 pass - last update
over 1 year ago 29,388 pass - last update
over 1 year ago 29,395 pass - last update
over 1 year ago 29,399 pass - last update
over 1 year ago 29,393 pass, 2 fail - last update
over 1 year ago 29,399 pass - last update
over 1 year ago 29,400 pass - last update
over 1 year ago 29,404 pass, 4 fail - last update
over 1 year ago Build Successful - Status changed to Needs work
over 1 year ago 4:15pm 7 June 2023 - 🇺🇸United States Amber Himes Matz Portland, OR USA
Looks like this MR needs a rebase, as the merge is blocked. It might work to just click the Rebase button in the MR link.
- last update
over 1 year ago 29,418 pass - Status changed to RTBC
over 1 year ago 4:33pm 7 June 2023 - last update
over 1 year ago 29,434 pass - last update
over 1 year ago 29,434 pass - last update
over 1 year ago 29,434 pass - last update
over 1 year ago 29,429 pass, 2 fail - last update
over 1 year ago 29,437 pass - last update
over 1 year ago 29,445 pass - last update
over 1 year ago 29,449 pass, 1 fail - last update
over 1 year ago 29,499 pass - last update
over 1 year ago 29,499 pass - last update
over 1 year ago 29,508 pass - last update
over 1 year ago 29,553 pass - last update
over 1 year ago 29,553 pass 42:01 40:51 Running- last update
over 1 year ago 29,565 pass, 2 fail - last update
over 1 year ago 29,569 pass, 1 fail - last update
over 1 year ago 29,801 pass - last update
over 1 year ago 29,801 pass - last update
over 1 year ago 29,802 pass - last update
over 1 year ago 29,800 pass, 2 fail - last update
over 1 year ago 29,806 pass 57:00 53:21 Running- last update
over 1 year ago 29,815 pass - last update
over 1 year ago 29,815 pass - last update
over 1 year ago 29,822 pass - Status changed to Fixed
over 1 year ago 9:10pm 19 July 2023 - 🇬🇧United Kingdom catch
Looks good to me, will make more sense once we add the mysqli driver but chicken-and-egg with that.
Committed/pushed to 11.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.