etianen / django-watson

Full-text multi-table search application for Django. Easy to install and use, with good performance.
BSD 3-Clause "New" or "Revised" License
1.21k stars 129 forks source link

Make join work with uuid primary key models #169

Closed vesterbaek closed 8 years ago

vesterbaek commented 8 years ago

Typecast non-int primary key columns to ::text. This makes Watson work with with UUID primary key columns.

etianen commented 8 years ago

Looks good!

I use a more generic approach in django-reversion here: https://github.com/etianen/django-reversion/blob/master/reversion/models.py#L287

However, since this code is postgres-specific, your solution is both minimal and elegant!

etianen commented 8 years ago

I've just baked this into a new release - 1.2.2

vesterbaek commented 8 years ago

Cool, thanks for merging and your work with this project.