- Issue created by @balazswmann
- 🇭🇺Hungary balazswmann
Sorry, I forgot the warning about issue forks. Attaching a patch then for
4.x
. - 🇭🇺Hungary balazswmann
Here is a patch for
4.3.10
too, as the4.x
codebase has progressed since then. - First commit to issue fork.
- 🇩🇪Germany mkalkbrenner 🇩🇪
Thanks for the patch.
But I have several concerns.
In theory, drupal can run on a non-SQL database. So the code needs to check for the DB driver. In case of mysql, the current value of wait_timeout could be read and be part of the calculation.
In general, wouldn't it be better to simply re-open the connection instead of keeping it alive?
Maybe that could could be written independently from a specific database driver so that we don't need SQL.In general, you should create a PR against our github repository to get the patch tested.