Opened 9 years ago

Closed 9 years ago

Last modified 4 years ago

#25175 closed Cleanup/optimization (fixed)

Create an alias for the postgresql_psycopg2 database backend

Reported by: Baptiste Mispelon Owned by: Caio Ariede
Component: Database layer (models, ORM) Version: 1.8
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

The name is too complicated and personally, I can never get it right the first time.

In the discussion on the mailing list [1], it was suggested to actually create two aliases: postgresql and postgres. I have no preference one way or the other.

[1] https://groups.google.com/forum/#!topic/django-developers/o7qFItlHILc

Change History (12)

comment:1 by Aymeric Augustin, 9 years ago

Given that Postgres was renamed to PostgreSQL a few years ago, I think that postgresql is a better name.

comment:2 by Wim Feijen, 9 years ago

Triage Stage: UnreviewedAccepted

As all other database names are named according to their database, I agree with Aymeric that postgresql is a better name.

comment:3 by Caio Ariede, 9 years ago

Owner: changed from nobody to Caio Ariede
Status: newassigned

comment:5 by Tim Graham, 9 years ago

Has patch: set
Patch needs improvement: set

Please don't forget to check "Has patch" so the ticket appears in the review queue.

As I mentioned on the pull request, I think we could leave the alias without an immediate deprecation. Mainly because I don't want to have to vary the settings on Jenkins (which are used for multiple versions of Django) based on the Django version. I guess it would save third-party apps some hassle too.

comment:6 by Tim Graham <timograham@…>, 9 years ago

Resolution: fixed
Status: assignedclosed

In ec900472:

Fixed #25175 -- Renamed the postgresql_psycopg2 database backend to postgresql.

comment:7 by Tim Graham <timograham@…>, 9 years ago

In bcb4fe00:

Refs #25175 -- Added backwards compatibility for importing postgresql_psycopg2 backend.

comment:8 by GitHub <noreply@…>, 8 years ago

In 3f0c4fe:

Refs #25175 -- Deprecated db.backends.postgresql_psycopg2 module.

comment:9 by Tim Graham <timograham@…>, 6 years ago

In 94446993:

Refs #25175 -- Removed db.backends.postgresql_psycopg2 per deprecation timeline.

comment:10 by Tim Graham <timograham@…>, 6 years ago

In ddb29368:

Refs #25175 -- Renamed test file referencing the old postgresql_psycopg2 engine.

comment:11 by Tim Graham <timograham@…>, 6 years ago

In 6cdc7ae:

[2.2.x] Refs #25175 -- Renamed test file referencing the old postgresql_psycopg2 engine.

Backport of ddb293685235fd09e932805771ae97f72e817181 from master

comment:12 by GitHub <noreply@…>, 4 years ago

In 5bcba16c:

Refs #25175 -- Removed postgresql_psycopg2 in django.db.utils.load_backend().

Follow up to 944469939b9eb93fda0924f78faba5c0ffae2dff.

Note: See TracTickets for help on using tickets.
Back to Top