search

exasol / django-exasol

Dialect for integrating EXASOL into the Django framework
https://www.djangoproject.com/
0 stars 2 forks source link

Remove dependency on django_pyodbc #15

Open frschwab opened 7 years ago

frschwab commented 7 years ago

Hi, why is this dependency necessary in django-exabackend/django_exabackend/base.py: https://github.com/EXASOL/django-exabackend/blob/f5e13154b5b5d52a9727835b872b6444568b296e/django_exabackend/base.py#L21 from django_pyodbc.compat import binary_type, text_type, timezone This shouldn't be used.

frschwab commented 7 years ago

This is also misleading in django-exabackend/setup.py:

setup( name='django-exabackend', long_description="EXASOL DB backend for Django", ... url='https://github.com/aurorasoftware/django-pyodbc', ... )

frschwab commented 7 years ago

I tried to just remove the dependency, but this results in errors, as binary_type, text_type and timezone are still part of the code. The question is: Can we rewrite the code to not use these dependencies, or do we have to live with the dependency?