Opened 5 years ago

Closed 5 years ago

Last modified 17 months 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 (11)

comment:1 Changed 5 years ago by Aymeric Augustin

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

comment:2 Changed 5 years ago by Wim Feijen

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 Changed 5 years ago by Caio Ariede

Owner: changed from nobody to Caio Ariede
Status: newassigned

comment:5 Changed 5 years ago by Tim Graham

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 Changed 5 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: assignedclosed

In ec900472:

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

comment:7 Changed 5 years ago by Tim Graham <timograham@…>

In bcb4fe00:

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

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

In 3f0c4fe:

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

comment:9 Changed 18 months ago by Tim Graham <timograham@…>

In 94446993:

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

comment:10 Changed 17 months ago by Tim Graham <timograham@…>

In ddb29368:

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

comment:11 Changed 17 months ago by Tim Graham <timograham@…>

In 6cdc7ae:

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

Backport of ddb293685235fd09e932805771ae97f72e817181 from master

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