Opened 6 years ago

Closed 6 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.

Subtickets

Attachments (1)

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

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by Christopher Allan Webber

Milestone: 0.7.0

comment:2 Changed 6 years ago by Christopher Allan Webber

Priority: majorcritical

comment:3 Changed 6 years ago by Jorge Araya Navarro

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 Changed 6 years ago by Jessica Tallon

Owner: set to Jessica Tallon
Status: newin_progress

Changed 6 years ago by Jessica Tallon

Attachment: issue_894.patch added

comment:5 Changed 6 years ago by Jessica Tallon

Owner: Jessica Tallon deleted
Resolution: fixed
Status: in_progressclosed

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.