Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#26285 closed Cleanup/optimization (fixed)

Deprecate the MySQL specific __search lookup

Reported by: Tim Graham Owned by: nobody
Component: Database layer (models, ORM) Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Found the implementation in Marc Taymlyn's branch for PostgreSQL full text text. It should have a separate ticket.

Change History (4)

comment:1 Changed 5 years ago by Tim Graham

Has patch: set

PR. As noted there, a remaining problem is that the __search lookup is used by the admin
for the search_fields = ['@fieldname'] syntax. I think #26184 would allow removing that code.

If we don't want to make that ticket a prerequisite of this one, I suppose we could leave that code for now, even if the search lookup is removed from Django itself.

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

Resolution: fixed
Status: newclosed

In 8ddc79a:

Fixed #26285 -- Deprecated the MySQL-specific search lookup.

comment:3 Changed 4 years ago by Tim Graham <timograham@…>

In 5139832:

Refs #26285 -- Removed MySQL search lookup per deprecation timeline.

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

In 0a0c245:

Refs #26285 -- Removed postgresql's unused DatabaseOperations.fulltext_search_sql().

Should have been removed in 5139832398624be75ee5361a6fac9348fdb61093.

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