- Issue created by @dpi
- 🇦🇺Australia dpi Perth, Australia
Added `suggest` until this is resolved: https://git.drupalcode.org/project/sm/-/merge_requests/14
- First commit to issue fork.
- Merge request !22Draft: Issue #3390775: Provide a Drupal SQL transport → (Closed) created by lussoluca
- 🇮🇹Italy lussoluca Italy
@dpi can we close this issue and add its scope to ✨ Implement retry Active ?
- 🇦🇺Australia dpi Perth, Australia
Still getting through a review.
It would be good to get this in before retry? are they tied together?
Do we need equivalents of doctrine_ping_connection / doctrine_close_connection / doctrine_open_transaction_logger for Drupal?
- 🇦🇺Australia dpi Perth, Australia
I've worked on a equivalent for doctrine_ping_connection & doctrine_close_connection
Looking to push changes including my not yet posted review comments after getting it going on a project. Stay tuned.
- 🇮🇹Italy lussoluca Italy
> It would be good to get this in before retry? are they tied together?
Yes, I think they should be worked together
> I've worked on a equivalent for doctrine_ping_connection & doctrine_close_connection
Cool, thanks!
- 🇦🇺Australia dpi Perth, Australia
The new work is found at https://git.drupalcode.org/project/sm/-/merge_requests/29 There is addressable feedback there. It's almost ready.
This MR precedes the work for ✨ Implement retry Active .
I don't see why they should be combined, so I've taken time to break them apart.
- 🇮🇹Italy lussoluca Italy
It seems that I don't have write permissions to drupal-sql branch.
I've created a new fork with the code from drupal-sql where I can add my commits (https://git.drupalcode.org/project/sm/-/merge_requests/36) - 🇮🇹Italy lussoluca Italy
MR https://git.drupalcode.org/project/sm/-/merge_requests/36 is ready to be reviewed
