Opened 10 years ago

Closed 10 years ago

#894 closed defect (fixed)

index User.username

Reported by: Christopher Allan Webber Owned by:
Priority: critical Milestone: 0.7.0
Component: programming Keywords: bitesized
Cc: Parent Tickets:

Description

We should add an index on User.username because, well, not having an index seems pretty crazy :)

Adding this index and migration shouldn't be hard, so marking as bitesized.

Attachments (1)

issue_894.patch​ (1.7 KB ) - added by Jessica Tallon 10 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by Christopher Allan Webber, 10 years ago

Milestone: → 0.7.0

comment:2 by Christopher Allan Webber, 10 years ago

Priority: major → critical

comment:3 by Jorge Araya Navarro, 10 years ago

By "adding this index" you mean, something like index=True as described ​here? or maybe I'm missing something.

I'm also new to sqlalchemy-migrate, so, I don't either know what "adding ... migration" means...

There is no documentation about the models, though...

comment:4 by Jessica Tallon, 10 years ago

Owner: set to Jessica Tallon
Status: new → in_progress

by Jessica Tallon, 10 years ago

Attachment: issue_894.patch​ added

comment:5 by Jessica Tallon, 10 years ago

Owner: Jessica Tallon removed
Resolution: → fixed
Status: in_progress → closed

This has been resolved (not by patch attached). The commit ​https://gitorious.org/mediagoblin/mediagoblin/commit/892eed590fb30131ea2e8612da5ba22fa24f690c has resolved this.

Note: See TracTickets for help on using tickets.