testing on postgres_psycopg2 tries to truncate tsearch2 tables

[shaunc <shaun@…>]

I'm using tsearch2 together with django. Tsearch2 adds some special tables to the template database which are automatically created in each new database. So they don't get mixed up or deleted, I leave the user as "postgres" -- not owned by the user that django connects via.

This works fine... except that during testing the (django) test cases "truncate" all tables including the tsearch2 tables. These tables shouldn't be truncated, and (since django is not the owner to prevent just this sort of thing) the command fails.

I have solved in patch by having django.db.backends.postgresql_psychopg2.introspection.get_table_list return by default only tables that are owned by the current_user. I think this is a reasonable default in general, because other tables are likely to be not the concern of django anyway.

• Shaun Cutts

[shaunc <shaun@…>]

patch of ...introspection.get_table_list, returning by default only tables owned by current user

[Yuri Baburov <burchik@…>]

Thanks! And what if my tsearch2 tables are owned by the same user like anything else?

[Adrian Holovaty]

I agree that Django should not be truncating -- or even *touching* -- the tsearch2 tables. But I would even go as far as to say that Django should not touch *any* tables that it doesn't know about, regardless of which user account the table was created under.

I will change this so that Django doesn't touch tables that aren't defined in models.

[Adrian Holovaty]

(In [6013]) Fixed #5086 -- The 'flush' and 'sqlflush' management commands no longer touch tables that Django is not aware of (tables that are not in INSTALLED_APPS and/or do not have associated models. Thanks for bringing this up, shaun@…

[GitHub <noreply@…>]

In f997b5e6:

Refs #5086 -- Removed unused only_django argument from sql_flush().

Unused (always True) since its introduction in 132605d889db8767a40243259066b8450428714c.