[PATCH 3/4] requirements: Switch to PyMySQL
Daniel Axtens
dja at axtens.net
Wed May 8 20:38:32 AEST 2019
Stephen Finucane <stephen at that.guru> writes:
> This is pure Python, which means there's no need for those pesky MySQL
> development libs.
>
I'm a bit worried about the performance impact of going from C to pure
Python: does have an impact on the speed of tests? Have you tested it
with a large parsearchive invocation?
Regards,
Daniel
> Signed-off-by: Stephen Finucane <stephen at that.guru>
> ---
> manage.py | 12 ++++++++++--
> requirements-dev.txt | 2 +-
> tox.ini | 2 +-
> 3 files changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/manage.py b/manage.py
> index fe6189ee..86bd67e9 100755
> --- a/manage.py
> +++ b/manage.py
> @@ -2,9 +2,17 @@
> import os
> import sys
>
> +try:
> + # From https://github.com/PyMySQL/PyMySQL/wiki/WhyPyMySQL
> + import pymysql
> + pymysql.install_as_MySQLdb()
> +except ImportError:
> + pass
> +
> +
> if __name__ == "__main__":
> - os.environ.setdefault("DJANGO_SETTINGS_MODULE",
> - "patchwork.settings.production")
> + os.environ.setdefault(
> + "DJANGO_SETTINGS_MODULE", "patchwork.settings.production")
>
> from django.core.management import execute_from_command_line
>
> diff --git a/requirements-dev.txt b/requirements-dev.txt
> index 0efdf886..06eefd99 100644
> --- a/requirements-dev.txt
> +++ b/requirements-dev.txt
> @@ -6,5 +6,5 @@ django-filter==1.1.0; python_version < '3.0' # pyup: ignore
> django-debug-toolbar==1.11
> django-dbbackup==3.2.0
> psycopg2-binary==2.8.2
> -mysqlclient==1.3.14
> +PyMySQL==0.9.3
> -r requirements-test.txt
> diff --git a/tox.ini b/tox.ini
> index ba5d5104..86fb62e1 100644
> --- a/tox.ini
> +++ b/tox.ini
> @@ -15,7 +15,7 @@ deps =
> django{20,21}: djangorestframework>=3.7,<3.10
> django{20,21}: django-filter>=2.0,<3.0
> postgres: psycopg2-binary==2.7.7
> - mysql: mysqlclient==1.3.14
> + mysql: PyMySQL==0.9.3
> docker =
> postgres: postgres:9.6
> mysql: mysql:5.7
> --
> 2.20.1
More information about the Patchwork
mailing list